Raymo111 / i3lock-color

The world's most popular non-default computer lockscreen.
Other
1.07k stars 133 forks source link

Please add an option to specify the circle radius #14

Closed JaneSmith closed 7 years ago

JaneSmith commented 7 years ago

It would make a huge difference to me if there was a simple command line option to specify the radius of the circle. Currently this seems to be hardcoded in as #define BUTTON_RADIUS.

For my resolution, the default radius makes the unlock indicator look very tiny on my screen, which is not pleasing to the eye. Making it customisable would also give me more freedom to align the unlock indicator with a background image.

PandorasFox commented 7 years ago

I've been poking at it some and it'll require more work. For now, the really hacky stuff I just did in 5 minutes is up here.

It works, but the text isn't aligned at all.

Eventually, I want to break out the ricing/drawing options into a config file, and leave the arguments for things like nofork/beep/show failed attempts. Ideally there'd be a way to just specify themes and look at some XDG dirs.

If I get that far, it wouldn't be difficult to just read in more stuff from a config file for text positioning and stuff. For now, specifying more arguments to handle things is... rapidly getting gross, and something I'm trying to avoid.

PandorasFox commented 7 years ago

Implemented in 6ec7c668cfc101b89076aa6226714397a66065a1