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

Properties name cut-off #1842

Closed
benedeki opened this issue Jul 7, 2021 · 2 comments · Fixed by #1845
Closed

Properties name cut-off #1842

benedeki opened this issue Jul 7, 2021 · 2 comments · Fixed by #1845
Assignees
Labels
bug Something isn't working Menas Menas component affected priority: high Critical to the health of the project UX Something concerning UX

Comments

@benedeki
Copy link
Collaborator

benedeki commented Jul 7, 2021

Describe the bug

A dataset property name longer than to fit into the give space is used - which is not very long - the rest of the name is cut off marked with an ellipsis. This doesn't help comprehension at all.

Expected behaviour

Use more space for the label containing the property name. Consider autoresizing the window to fit the whole name,

Screenshots

image

@benedeki benedeki added bug Something isn't working Menas Menas component affected UX Something concerning UX priority: high Critical to the health of the project labels Jul 7, 2021
@benedeki
Copy link
Collaborator Author

benedeki commented Jul 7, 2021

Idea/suggestion:
As resizing the parent window based on the length of the label might be complicated.
What about creating a group box with a caption Dataset properties:
Inside would then be properties using the whole width of the window. Furthermore if this extra space is not enough, in case the label is longer, let's put it on its own line, while the edit box and icon bellow it.
And even if that is not enough - IMHO clearly misused name - the whole name could be put into the hint when hovering over the label.

dk1844 added a commit that referenced this issue Jul 8, 2021
…raged - removed hard limits (min width, enabled resizing to accommodate any needs with sensible defaults)
@dk1844 dk1844 linked a pull request Jul 9, 2021 that will close this issue
dk1844 added a commit that referenced this issue Jul 12, 2021
…raged - removed hard limits (min width, enabled resizing to accommodate any needs with sensible defaults)
@dk1844
Copy link
Contributor

dk1844 commented Jul 15, 2021

Release notes
Dataset properties editing UX has been enhanced - now, there is more room for the property name, a useful tooltip is provided and the dialog is resizable (Small by default).

dk1844 added a commit that referenced this issue Jul 15, 2021
* #1842 - DS Properties on DS dialog: layout tweaked, resizable (default: Small), enhanced tooltips, properties border
dk1844 added a commit that referenced this issue Jul 15, 2021
* #1842 - DS Properties on DS dialog: layout tweaked, resizable (default: Small), enhanced tooltips, properties border
dk1844 added a commit that referenced this issue Jul 16, 2021
* #1842 - DS Properties on DS dialog: layout tweaked, resizable (default: Small), enhanced tooltips, properties border
dk1844 added a commit that referenced this issue Jul 16, 2021
* #1842 - DS Properties on DS dialog: layout tweaked, resizable (default: Small), enhanced tooltips, properties border
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Menas Menas component affected priority: high Critical to the health of the project UX Something concerning UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants