Skip to content
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

[BUG] Recent changes on default gnome-shell theme make password entry box to be hardly readable #176

Closed
pacho2 opened this issue Aug 14, 2022 · 7 comments · Fixed by #189
Labels

Comments

@pacho2
Copy link

pacho2 commented Aug 14, 2022

Describe the bug
With latest gnome-shell default theme updates (I think they were introduced in Gnome 41 cycle), the password entry box was made transparent, that makes the box hardly readable when the extension is configured to get a fully transparent lock screen and a brilliant wallpaper is used.

Maybe the entry box theme could be tweaked back to safer transparency/color options :/

Thanks a lot

Desktop (please complete the following information):

  • OS: Gentoo Linux
  • Extension Version: 41
  • GNOME Version: 42
@pacho2 pacho2 added the bug label Aug 14, 2022
@pacho2 pacho2 changed the title [BUG] [BUG] Recent changes on default gnome-shell theme make password entry box to be hardly readable Aug 14, 2022
@neffo
Copy link
Owner

neffo commented Aug 29, 2022

Yeah, it's not really ideal. Ultimately the extension isn't actually running when the lock-screen is active (that is possible but requires more hoops to jump through to get approval). I'm thinking of a few ideas but haven't come close to prototyping it. One option would be to split the extension so that we do the blur (and whatever features we want), but keep the code clean and easier to audit which is important for a lock-screen. Then it might be possible to inject a blurred backdrop to the login dialogs. Just an idea atm though.

@pacho2
Copy link
Author

pacho2 commented Dec 22, 2022

Maybe blur could be kept for the prompt screen. It seems it is the way gnome-shell-extension-unlockDialogBackground is handling this problem 🤔
https://github.com/sunwxg/gnome-shell-extension-unlockDialogBackground/blob/master/unlockDialogBackground%40sun.wxg%40gmail.com/extension.js#L74

@neffo
Copy link
Owner

neffo commented Dec 23, 2022

That is a good idea, yeah.

In this case, it might be better for me to just advise people use the unlock dialog extension and remove this code from Bing Wallpaper.

@pacho2
Copy link
Author

pacho2 commented Dec 23, 2022

But that extension is different, is not as simple as https://github.com/PRATAP-KUMAR/control-blur-effect-on-lock-screen , it forces the user to set a "hard coded" image to be shown in the lock screen. Indeed, if I enable both, unlockdialogbackground extension will simply make lock screen to show a blue background is I don't manually set an image

neffo added a commit that referenced this issue Jan 5, 2023
neffo added a commit that referenced this issue Jan 5, 2023
@neffo
Copy link
Owner

neffo commented Jan 5, 2023

Ok I have a working implementation of this now, feel free to test #189 if you are adventurous.

I'm comfortable with what we are doing, but two things of note.
Firstly, there's no transition, so it's a particularly abrupt (not Windows-like).

Secondly, we need to keep the Blur object active for this to work, which means we aren't disabling every part of the extension when the lockscreen is active. This isn't 100% compatible with EGO review guidelines seen here. We could explicitly ask for the whole extension to remain active during both session mode in metadata, but would be problematic. An alternative would be to have this part of the extension separate from Bing Wallpaper. (It's something I considered, and may be forced when it gets reviewed.)

@neffo neffo linked a pull request Jan 5, 2023 that will close this issue
8 tasks
@neffo
Copy link
Owner

neffo commented Jan 5, 2023

Testing on Fedora:
image
image

neffo added a commit that referenced this issue Jan 13, 2023
* update french translation (#188)
* adjustable icon size for menu controls #192
* GNOME default blur for login prompt #176
* code clean ups and refactoring
* streamline menu and improve ui/ux

Co-authored-by: 0xMRTT <0xMRTT@proton.me>
@pacho2
Copy link
Author

pacho2 commented Jan 24, 2023

Sorry for the late reply, it works fine!

Thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants