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
breeze-plymouth: add optional nixos branding, enable for plasma5 #31378
Conversation
Thank you! Looking at the implementation of the breeze theme, I can see why you didn't make the branded variant independent from the breeze theme even though on the surface there is only the spinner in common. In the end though it may be better to extract it into a separate repository (if the breeze theme is independent from However, the current label under the logo is not useful or pretty. For me it reads There is also a tiny issue that both commits add some spaces at the ends of the lines. It is better to configure your editor to automatically trim whitespace. |
The problem with making it a separate derivation is that if you want to use the nixos branded one with a version, you need to be in a nixos context (like a module) where you have access to the version. So it seems like it has to be defined by an override in a module. I did think about making it the default plymouth theme (IMO it's a lot nicer than the current default one), do you think I should just go ahead and do that? I think I thought that the |
4e610c5
to
0470221
Compare
Updated to use I think I'll open a separate PR to suggest making this the default, and possibly also enabling it on the live CD to make a good first impression. |
Gentle ping @orivej . Are you happy with this now? |
0470221
to
eec4c31
Compare
Thanks!
…On 18 Nov 2017 10:37, "Orivej Desh" ***@***.***> wrote:
Merged #31378 <#31378>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#31378 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABnuioeBHuk_N_Ie5jGr_T2udipMm2Hgks5s3rNpgaJpZM4QVot->
.
|
Thank you too! |
This is probably for another issue, but when I enable this theme, my laptop hangs on boot to the point when it's completely unresponsive (ESC key doesn't work so I can't figure out what went wrong). If I hit ESC a bit earlier (before it hangs) and hide Plymouth, everything is fine. |
Motivation for this change
I noticed that the breeze-plymouth theme has some options for customizing it for a distro. I thought it would be nice to do this for nixos.
I then enabled this for the version that's used by plasma5 by default.
Note that we need to set some of the arguments in an override directly in the nixos module, since we want to include the vesion.
The result is now quite a nice splash screen which includes the proper nixos version and logo. There are also some options to set a color gradient in the background, but the selection of colors is quite limited and I wasn't sure any of it looked better than just black, so I left it.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)