-
-
Notifications
You must be signed in to change notification settings - Fork 225
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
Feature: Glanceable UI theme + additional progress percentage. #1151
Feature: Glanceable UI theme + additional progress percentage. #1151
Conversation
Filament screen corrected with the following changes: * Filament bar background colour changed to match theme. * Filament table background gradient removed. * Filament bar width adjusted to match files section, this also lined up the scroll bar with its background.
Addition of a second progress percentage which is attached to the progress bar div. CSS set to make this percentage hidden by default. This will allow the creation of themes where percentage is visible for both round and straight progress bars.
Theme creates a horizontal progress bar when printing that is more easily readable from across the room. This theme requires the additional div from the previous commit, but the instructions in the Readme.md include a workaround.
Very curiously it appears it's trying to remerge c2822a2... I don't understand why since the lines it wants to merge are already part of the NOX theme. Also please excuse the "Merge branch 'master' " messages. It took me a while to realise that Visual Studio Code wasn't rebasing. Let me know if you want me to nuke this completely and start again or if this is able to be merged... |
Should work now without an additional percentage diff. Let me know if everything works as you expect. |
Cheers I'll have a look tonight if this achieved what was intended. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works great. I didn't know about the "+" element selector in CSS. This solution is much neater than an additional element.
Thanks.
Yeah I also just learned about that selector a week or so ago. |
…rtedBull#1151) * Updated Filament in Theme. Filament screen corrected with the following changes: * Filament bar background colour changed to match theme. * Filament table background gradient removed. * Filament bar width adjusted to match files section, this also lined up the scroll bar with its background. * Progress Percentage Added Addition of a second progress percentage which is attached to the progress bar div. CSS set to make this percentage hidden by default. This will allow the creation of themes where percentage is visible for both round and straight progress bars. * New Theme: "Glanceable" Theme creates a horizontal progress bar when printing that is more easily readable from across the room. This theme requires the additional div from the previous commit, but the instructions in the Readme.md include a workaround. * Cleaned up whitespace at EOL * remove third percentage div * Slightly move percentage in progress bar to left. Co-authored-by: UnchartedBull <timon.gaebelein@icloud.com>
This PR further addresses issue #749 for a more progress centric UI.
It includes a new "Glanceable" theme which is a variant on the standard theme but with a horizontal progress bar.
Additionally a second progress percentage element has been added to the app nested with the horizontal progress bar. This allows a progress percentage to be visible and in two different places on the screen depending on which progress bar is shown. The CSS is setup to ensure this second progress bar is hidden unless enabled by a theme.
This change was tested against both the default theme as well as the NOX theme to ensure it doesn't break either.