Bug 501606 - New URL/Location Bar background does not indicate edit state (too light/dark)
Summary: New URL/Location Bar background does not indicate edit state (too light/dark)
Status: RESOLVED NOT A BUG
Alias: None
Product: frameworks-kio
Classification: Frameworks and Libraries
Component: URL navigator (show other bugs)
Version: 6.12.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KIO Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-03-16 21:28 UTC by Blazer Silving
Modified: 2025-03-20 15:32 UTC (History)
9 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Blazer Silving 2025-03-16 21:28:06 UTC
SUMMARY
Please refer to my feedback on https://invent.kde.org/frameworks/kio/-/merge_requests/1793#note_1170629
RE: https://bugs.kde.org/475106

STEPS TO REPRODUCE
1. Open Dolphin and navigate to a folder
2. Location Bar background is shaded now (lighter shade in light mode, darker in dark)
3. However, clicking to edit the bar does not change the background shade as before. 

OBSERVED RESULT
The Location Bar seems "stuck" in edit mode, when it's not. Other indicators are the only way to know.

EXPECTED RESULT
The Location Bar should be visibly different when editing the text field and when not. 

SOFTWARE/OS VERSIONS
KDE Plasma Version: 6.3.3
KDE Frameworks Version: 6.12
Qt Version: 6.8.2

ADDITIONAL INFORMATION
I brainstorm this could be resolved by introducing a new colorscheme value to use for the default URL bar state, only a slight slight shade difference from the header. Not sure if this value could be calculated dynamically, but having it as a colorscheme value seems like the best solution. 

Personally, I much preferred when the default location bar matched the header as it has for years. A slight shade of a difference could change my mind, but if given the option to define the color in my preferences, I will likely set it back to match the header exactly.
Comment 1 Méven 2025-03-17 09:52:01 UTC
> The Location Bar should be visibly different when editing the text field and when not.

Excluding the background color, it is the same for all other visual clues as before

In editing mode, there are many hints that we are in editing path mode:
The icon at the beginning of the path (instead of >), the cursor, the '/' visible, the <X to remove, the V confirmation...

It is more subtle now. I suspect this is mostly a transition phase for something you were visually accustomed to.

Still we might adjust this somewhat, not clear how.
We don't want to use the toolbar background color, maybe a 1.5 lighter version of it.
Comment 2 Blazer Silving 2025-03-17 16:39:03 UTC
> In editing mode, there are many hints that we are in editing path mode:
The icon at the beginning of the path (instead of >), the cursor, the '/' visible, the <X to remove, the V confirmation...

Yes, I briefly note that in my initial feedback: Now that the backgrounds share the same color, the ONLY way to tell if you are in edit mode is to be cognizant of the other cues (for me the most visible is the check icon at far end of the bar, requiring scanning the bar to find the icon. 

> I suspect this is mostly a transition phase for something you were visually accustomed to.
A color to indicate the normal state is now missing when it was functional before. This is a regression. 

The most clear solution would be to use an adjacent shade of the toolbar header for the default bar state: not the exact shade, and not even a 1.5 mix with the edit background (too strong). 

It will take me some time to draw this up.
Comment 3 Blazer Silving 2025-03-17 17:04:12 UTC
It's also worth noting the "sunken" state of the bar in all cases now. The bar was formerly in the same plane as the header while in normal mode, and would "sink" into edit mode (looked great). 

Simply raising the Location Bar (either at rest or on hover) in this same axis would communicate the ability to interact without requiring color work. The bug report that kicked off this entire effort was only requesting this change in the first place.
Comment 4 Blazer Silving 2025-03-19 19:24:33 UTC
Update, this MR addresses the background shade along with other fixes: https://invent.kde.org/frameworks/kio/-/merge_requests/1842#note_1172777

This also adds an internal toggle for the feature entirely, `m_backgroundEnabled`. While I don't think it's intended, this can be used to fully revert to the old presentation. 

This is great, but I feel the default contrast should be be stronger, and I still prefer the background to fully match the header/toolbar. I have a few screenshots of what I think would be the best mix for those that prefer a background: https://invent.kde.org/frameworks/kio/-/merge_requests/1842#note_1172684
Comment 5 TraceyC 2025-03-20 15:32:58 UTC
The best place to discuss feedback on merge requests is in the MR themselves. I'm going to close this report out since it isn't really reporting a bug, but discussing feedback on those MRs. Thanks for your understanding.