Skip to content

Conversation

@dharit-tan
Copy link

@dharit-tan dharit-tan commented Feb 19, 2025

kwvg and others added 30 commits April 21, 2023 00:47
git-subtree-dir: extensions/les
git-subtree-mainline: c9d612e
git-subtree-split: 24cf433
kwvg and others added 25 commits July 7, 2023 11:37
This should be a good enough replacement for upgrade logic where users
were expected to throw away the settings.ini they had, copy the one
pre-bundled and then modify it to match the same behavior as earlier.

This should be additive in nature, not subtractive
This is to prepare the stage for removing the pre-baked settings.ini
file altogether. This check is too early, before there is an attempt
to read (and therefore generate) a valid settings.ini so it'll try to
substitute a pair in an empty file...

Let LES write default values, then prompt the user.
One of Lua's pitfalls is that everything can be a global by default, it
makes understanding the initialization process a bit of a PITA. To work
despite that, we're trying to make the best of the pseudo-OOP that Lua
provides.

We'll slowly try to streamline and document the initialization sequence,
changing it if we have to.

For instance, we're doing the settings read _before_ the macOS and Live
sanity check and _well before_ we're reading the menuconfig. The reason
will be apparent in subsequent commits.
§ is only available on certain European layouts, ` is available on most
layouts, including the more popular US layout. It makes sense for a sane
default to be based on the US layout rather than the European one.
@kwvg
Copy link
Member

kwvg commented Feb 20, 2025

Thank you for your contribution!

I'd have to borrow a Mac to compile and test the fixes, that's primarily why code contributions on my end haven't happened in a while but in the meanwhile, is it possible to get in contact with the author of the fix so that they can give permission to use it?

@Alexgreshammusic
Copy link

Alexgreshammusic commented Feb 20, 2025

Hey this is Orphin on Reddit
Feel free to use this adjustment.

this fix has worked on my M1 Pro Chip
My 2018 MacBook Pro with touchbar
And my custom Hackintosh
(6900xt I9 9900x)

@kwvg
Copy link
Member

kwvg commented Feb 21, 2025

Thank you!

Am currently fixing up a Mac, will be rebasing on top latest Hammerspoon (1.0.0 vs current base of 0.9.99) and improving Live version detection (reading from Info.plist instead of going off the name of the app bundle) so that the fix can be incorporated while retaining logic for Live 11 and earlier. Though, am busy this week so this might take some time.

@kwvg
Copy link
Member

kwvg commented Feb 22, 2025

@dharit-tan develop has been updated to be based on Hammerspoon 1.0.0 which unfortunately has caused history divergence, plus, additional changes had to be made to allow for retaining the keystroke pattern for Live 11 and earlier.

Please consider basing your PR on pr_52 (which incorporates your fixes as cd43968, 4aba4fe and a813633).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants