Add option to separate full path visibility for tab and window.
AbandonedPublic

Authored by aprcela on Apr 13 2020, 11:25 AM.

Details

Reviewers
None
Group Reviewers
VDG
Dolphin
Summary

Add option to have the full path visible on either Tab, Window or both

Diff Detail

Repository
R318 Dolphin
Branch
TabNameFullPath (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25154
Build 25172: arc lint + arc unit
aprcela created this revision.Apr 13 2020, 11:25 AM
Restricted Application added a project: Dolphin. · View Herald TranscriptApr 13 2020, 11:25 AM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
aprcela requested review of this revision.Apr 13 2020, 11:25 AM
aprcela edited the summary of this revision. (Show Details)Apr 13 2020, 11:27 AM
aprcela edited the summary of this revision. (Show Details)

I wonder... might it make sense to just always display the full path in the titlebar? And as for tab titles, does it ever make sense to display the full path there? The amount of horizontal space is very small so the path will almost always be elided away into uselessness.

aprcela added a comment.EditedApr 13 2020, 4:19 PM

I wonder... might it make sense to just always display the full path in the titlebar? And as for tab titles, does it ever make sense to display the full path there? The amount of horizontal space is very small so the path will almost always be elided away into uselessness.

Yup, that's exactly the setup I would use. Current settings allow only "full path everywhere, or nowhere". This allows the user to set it up as they want it. Showing the full path always in the window title and tab only current folder - I'd say go for it. But would every user prefer that? And... ain't KDE famous for the overwhelming amount of options? :)

Also, is the way how I implemented this ok?

I wonder... might it make sense to just always display the full path in the titlebar? And as for tab titles, does it ever make sense to display the full path there? The amount of horizontal space is very small so the path will almost always be elided away into uselessness.

Yup, that's exactly the setup I would use. Current settings allow only "full path everywhere, or nowhere". This allows the user to set it up as they want it. Showing the full path always in the window title and tab only current folder - I'd say go for it. But would every user prefer that? And... ain't KDE famous for the overwhelming amount of options? :)

Yeah, but we're trying to move away from that and toward having good defaults and offering only options that make sense. :)

Also, is the way how I implemented this ok?

The implementation looks sane, I'm just wondering if we should instead make the tab never display the full path (because it never makes sense there) and make the option only affect the window's titlebar.

VDG folks, your thoughts?

Yeah, showing the full path in the tab makes very little sense, so we shouldn't allow it.

And -1 for adding yet another option, we already have two "Show full path" settings and that's already too much.

I wonder... might it make sense to just always display the full path in the titlebar?

Just about this specific part, I wouldn't recommend it, because not every path is as "user friendly" as a full local path.
There's even a bug report about it: https://bugs.kde.org/show_bug.cgi?id=406624. Example:
https://bugs.kde.org/show_bug.cgi?id=406624

Maybe the "full path" heuristics should be improved before taking that step.
This is also a problem in the history menu (which does not follow any setting and shows always a "full path")

Yeah, but we're trying to move away from that and toward having good defaults and offering only options that make sense. :)

Ah okay, a lil housekeeping then. Just change or remove the current option.

Yeah, showing the full path in the tab makes very little sense, so we shouldn't allow it.

And -1 for adding yet another option, we already have two "Show full path" settings and that's already too much.

So I can disregard this and maybe change the current option "Show full path in titlebar" to only switch the full path On/Off for the window title instead of both window and tab?

So I can disregard this and maybe change the current option "Show full path in titlebar" to only switch the full path On/Off for the window title instead of both window and tab?

Yep, makes sense to me!

aprcela abandoned this revision.Apr 14 2020, 10:04 AM

So I can disregard this and maybe change the current option "Show full path in titlebar" to only switch the full path On/Off for the window title instead of both window and tab?

Yep, makes sense to me!

Ok.
I'm abandoning this one.