-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
i3lock-color is out of date compared to i3lock #6
Comments
For anyone interested, I've rebased i3lock-color on current day i3lock here. I also fixed a couple bugs in the original i3lock-color. |
Hey, thanks for posting here and reminding me this repo still exists, and sorry for not replying for so long! I actually haven't looked at this repo in a while (I've been using 'standard' i3lock again). I'm actually surprised that people are still interested in this. Yes, it is out of date since it was more or less a proof-of-concept back then and I'm also not too happy with the way you have to specify the colors as lots of commandline arguments. @meskarune if you want to take over this project that's cool! I'm just not quite sure what's the best way to do this on github - adding you as a contributor, or deleting my repo after you forked it, or just transferring it, or do you just want to send me pull requests..? If you have any question on the code itself feel free to ask. As far as I remember the diff to the original i3lock is quite small anyway. However, from a quick look I think @Arcaena 's fork is the one that should be continued since it's not based on an outdated version of i3lock... |
yeah, the original diff compared to i3lock at the time was pretty small but it had changed enough since then that it was easier to just fork off of i3lock and then patch in the stuff, otherwise I'dve directly forked this since that would've been proper. That reminds me though, I actually did find a bug here that I fixed in my fork, should I do a pull request, or nah? |
Sure, send me a pull request. At the moment I'm thinking about also changing the readme so it directs people to your repository if they actually want to use i3lock-color. |
Alright, thanks! If you want to change the readme, that'd be awesome. |
OK, merged your PR and added the note. I'm closing that issue for now. Thanks again for bringing it up. And sorry to @meskarune that I didn't notice the issue earlier. I just didn't notice the notification at the time - but maybe you two can communicate and make i3lock even cooler. Also, there's two funny i3lock forks by @ravinrabbid who added SVG rendering and little clock hands in the circular indicator: https://github.com/ravinrabbid/i3lock-svg |
The i3lock package is ahead of i3lock-color by quite a bit. It would be amazing if i3lock-color could be updated with the latest i3lock. If you don't plan on maintaining this any longer I could perhaps try my hand, but I'm very new to C programming and may need some help knowing what exactly was changed to add the new features. This package is still widely used by many people and I would hate to see it get lost.
The text was updated successfully, but these errors were encountered: