Bug 475103 - Terms "Horizontal" and "vertical" are confusing in this context
Summary: Terms "Horizontal" and "vertical" are confusing in this context
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: Custom Tiling (show other bugs)
Version: 5.27.8
Platform: unspecified Linux
: NOR minor
Target Milestone: ---
Assignee: Nate Graham
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-01 17:51 UTC by holyzolly
Modified: 2024-05-20 16:08 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.1


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description holyzolly 2023-10-01 17:51:10 UTC
Sorry, can't find a good category.
SUMMARY
***
The terms
"Horizontal" vs "vertical"
"Left-Right" vs "Top-Bottom"
are confused in the options of the Tiling Layout creator (to be accessed by SUPER + T).
***


STEPS TO REPRODUCE
1. log into Plasma and hit SUPER + T
2. Take one of the fields and select: Split Vertically (why is it a capital V btw?) and it splits the field horizontally, i. e. the "split" is then a horizontal line.
3. Take one of the fields and select: Split Horizonatlly (why is it a capital H btw?) and it splits the field vertically, i. e. the "split" is then a vertical line.

OBSERVED RESULT
Vertical split is horizontal.
Horizontal split is vertical.

EXPECTED RESULT
Vertical split is vertical.
Horizontal split is horizontal.

SOFTWARE/OS VERSIONS
Manjaro Linux (Stable). KDE Plasam
KDE Plasma Version: 5 27 8
KDE Frameworks Version:  5 110 0
Qt Version: 5 15 10

ADDITIONAL INFORMATION
It's the same in Konsole with "Left-Right" vs "Top-Bottom". There might be other apps as well?
Comment 1 holyzolly 2023-10-01 17:59:48 UTC
Ok, sorry the Konsole option does make sence. Split view, which is adjective split; noun: view.
It wouldn't make sense if you interpret split as verb.

Maybe things get/got lost in translation :)
into other languages.
Comment 2 holyzolly 2023-10-01 19:14:06 UTC
Sorry pls delete <"Left-Right" vs "Top-Bottom"> from summary above.
Comment 3 hey 2024-04-26 17:38:56 UTC
(In reply to holyzolly from comment #0)
> It's the same in Konsole with "Left-Right" vs "Top-Bottom".

I actually like Konsole's phrasing and would love to have it in the tiling editor as well. "Split view left-right" means, to me, that I will have a left window and a right window in the end, which is precisely what I get. I think it's perfectly intuitive.
Comment 4 Karl Ove Hufthammer 2024-05-16 18:29:31 UTC
I too find this very confusing. I would expect ‘Split Vertically’ to actually split the view vertically (into a left and a right part), just like the same option does in Kate, not horizontally (into a top and a bottom part), as in the current behaviour in the tiling manager. And ‘Split Horizontally’ to split to view horizontally, not vertically.

And it looks like I’m not the only one confused by the wording:
https://www.reddit.com/r/kde/comments/112gube/wow_how_did_they_miss_this_one_horizontal_runs/

I’m translating KDE, and this actually caused me to use the wrong translation. I translated ‘Split Vertically’ into something like ‘Split left/right’, as I thought this would be easier to understand, and since it was the the *correct* translation for the similarly named feature in Kate.
Comment 5 Nate Graham 2024-05-17 21:38:18 UTC
In progress with https://invent.kde.org/plasma/kwin/-/merge_requests/5748.
Comment 6 Nate Graham 2024-05-20 16:08:15 UTC
Git commit dcd2f7b15a310d2bdb8966c59b52652657337dc1 by Nate Graham.
Committed on 20/05/2024 at 15:58.
Pushed by ngraham into branch 'master'.

plugins/tileseditor: clarify wording in "split the view" buttons

Apparently people have differing ideas of what "vertically" and
"horizontally" mean, and some feel that this wording contradicts the
icon.

That's not accurate, but the wording is admittedly a little a bit
ambiguous. We can clarify this by using super duper explicit
terminology that always matches the icon.
FIXED-IN: 6.1

M  +2    -2    src/plugins/tileseditor/qml/TileDelegate.qml

https://invent.kde.org/plasma/kwin/-/commit/dcd2f7b15a310d2bdb8966c59b52652657337dc1