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

Draft: Display document status of other dimensions in dimension dropdown #3680

Open
wants to merge 12 commits into
base: 8.4
Choose a base branch
from

Conversation

pKallert
Copy link
Contributor

@pKallert pKallert commented Dec 28, 2023

What I did
In the single-dimensions dropdown, the Languages with existing documents are now a linked and can be opened in a new tab.
The dropdown now shows which dimensions the current already document exists in. "Not allowed" dimensions have a new styling

How I did it

For the dimensions menu there are two scenarios. This here describes the current implementation, but everything is still up for discussion

Single dimension
"Shine through" for en_UK and en_US
Bildschirmfoto 2024-01-02 um 11 00 58
=> English(US) and English(UK) can be opened in new tab

For "normal" languages: only one dimensions highlighted
Bildschirmfoto 2024-01-02 um 11 01 54

Multiple dimensions
(Currently) no shine through for dimensions.
No linking of dimensions, this is what the apply button is for, I think?
Disallowed dimensions are marked with strike-through (for now)
Bildschirmfoto 2024-01-02 um 13 50 49

Available dimensions are shown top-down.
For example, if in the first dropdown "Deutsch" is chosen as the language, in the second dropdown only the variants with language deutsch and country * will be chosen.
Bildschirmfoto 2024-01-02 um 13 57 05

ToDos

  • full width of links in dropdown
  • write e2e tests

To be discussed

  • Do we even want the "available" Elements shown in multiple dimensions? Is it understandable?
  • Visualize "Shine through" of languages in dropdown menu?
  • Are shine-through dimensions counted as "available"?
  • Better display of "disallowed" vs "available"?

@github-actions github-actions bot added the 8.4 label Dec 28, 2023
@markusguenther
Copy link
Member

This is a successor PR of #3597

@pKallert pKallert changed the title Draft: Add "non-existant" state to dimension Menu Draft: Display document status of other dimensions in dimension dropdown Dec 29, 2023
@markusguenther
Copy link
Member

Hello, and thanks to @pKallert for highlighting that fantastic feature. I've reviewed the code and experimented with it, and it functions as you outlined. However, I'm not particularly fond of the strikeout method for disallowed dimensions. With the introduction of the new grouping feature in UI version 8.4, I thought about utilizing groups for this purpose.

This approach could potentially enhance the user experience, as users would better comprehend the implications. We could incorporate a helpful text within the groups to explain the meaning of terms like "untranslated" or "disallowed." Additionally, it might be beneficial to consider hiding disallowed dimensions, as they occupy space without serving any practical purpose.

What do you and the others think about that?
Maybe a help message would be a nice feature extension for the groups anyway.

Screenshot 2024-01-02 at 19 54 09

@markusguenther
Copy link
Member

markusguenther commented Jan 2, 2024

Addendum: Linking to other dimensions in a new tab doesn't work for me in Firefox, Chrome and Safari

@pKallert pKallert force-pushed the feature/AddInactiveContentDimensionsDropdown branch from f642c52 to 7da44f2 Compare January 3, 2024 07:48
@pKallert pKallert force-pushed the feature/AddInactiveContentDimensionsDropdown branch from 7da44f2 to 76c0299 Compare January 3, 2024 07:49
@pKallert
Copy link
Contributor Author

I thought a little bit more about the styling of the elements and came up with a new solution:

Element does not exist in dimension: use grey background color
Bildschirmfoto 2024-01-30 um 08 58 50

Element is disallowed: Text color is grey
Bildschirmfoto 2024-01-30 um 15 00 40

Example:
Bildschirmfoto 2024-01-30 um 15 09 15
Francais-> France exists,
Francais-> Germany is not allowed, and does also not exist
Francais-> US is allowed, but does not exist

This would also maintain the current logic where disallowed Elements are grey/have opacity.

@markusguenther
Copy link
Member

markusguenther commented Jan 30, 2024

That's funny, I also had this topic yesterday on my desk to continue with that.
I came to this idea.

  1. Colored Bars:
    • Using a small colored bar for each entry is a concise and visually efficient way to communicate the status of dimensions.
    • The traffic light metaphor (red, orange, etc.) is intuitive and widely understood, making it user-friendly.
  2. Not Permitted Dimensions:
    • Greyed-out entries with a red bar make it clear that these dimensions are not permitted. This combination of visual cues effectively signals to users that these dimensions are restricted.
  3. Not Translated Dimensions:
    • Introducing an orange bar and a zebra background for dimensions that are not translated adds a distinct visual element to highlight this state.
    • The use of a 45-degree angle for the zebra background creates a visually interesting effect and sets it apart from the regular and greyed-out dimensions.

I am also half the way of the implementation. What do you think about that?

@pKallert
Copy link
Contributor Author

I tried it out locally and now have the following design:
Not permitted:
Bildschirmfoto 2024-03-15 um 08 19 37

Not translated:
Bildschirmfoto 2024-03-15 um 08 20 33
Bildschirmfoto 2024-03-12 um 11 46 46
For not tranlated we could also not do the striped background:

Bildschirmfoto 2024-03-12 um 12 41 01

As a follow-up we could also visualize the shine-through with fallback dimensions.

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 this pull request may close these issues.

FEATURE: Usability Improvements for Dimensions Menu
2 participants