Bug 575695 - Gitflow Start-Dialogs do not fully display long Error Messages
Summary: Gitflow Start-Dialogs do not fully display long Error Messages
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: Gitflow (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 minor (vote)
Target Milestone: 5.13   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-29 11:50 EDT by Max Hohenegger CLA
Modified: 2021-09-14 11:01 EDT (History)
2 users (show)

See Also:


Attachments
feature start dialog with error (12.33 KB, image/png)
2021-08-29 11:50 EDT, Max Hohenegger CLA
no flags Details
gtiflow start feature error still hidden (26.89 KB, image/png)
2021-09-14 11:01 EDT, Max Hohenegger CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Max Hohenegger CLA 2021-08-29 11:50:00 EDT
Created attachment 287042 [details]
feature start dialog with error

In case of input errors (e.g. invalid symbols), the dialog to start a feature branch would display an error message. If the branch-name gets very long, the end of the message would be cut off, and the message would not help the user resolve the problem.

See attached screenshot.
Comment 1 Eclipse Genie CLA 2021-08-29 11:53:45 EDT
New Gerrit change created: https://git.eclipse.org/r/c/egit/egit/+/184743
Comment 3 Thomas Wolf CLA 2021-09-02 04:01:46 EDT
That's an ugly work-around. Did you actually resize the dialog? At least on OS X, the layout is ugly; the message above the text input field grabs way too much space.

One way of improving this might be to omit the branch name from the error message; the user sees it anyway in the text input field. But even then the error message might still get too long, depending on what the validator returns.

A better way IMO is to make the dialog adjust its size automatically such that it can display the whole message. (Note that the error message label has SWT.WRAP set.)
Comment 4 Eclipse Genie CLA 2021-09-02 04:03:38 EDT
New Gerrit change created: https://git.eclipse.org/r/c/egit/egit/+/184907
Comment 5 Max Hohenegger CLA 2021-09-02 11:50:10 EDT
It looked ok on Ubuntu. Sure, you can resize it to a point where the layouting does not make sense anymore, but why would you want to do that?
If you are just interested in the full error message you'd just extend the dialog horizontally to make room for a few more words.
Could you share a screenshot of how it looks like in OS X?

I agree it's a workaround. I would've preferred inserting ellipsis in the branch name, but didn't want to spend that kind of effort, for what I think, is a corner-case.
I see you already provided a different solution. I'll try to have a look, but don't know yet when I'll get to it.
Comment 7 Max Hohenegger CLA 2021-09-14 11:01:44 EDT
Created attachment 287128 [details]
gtiflow start feature error still hidden

I think the new fix will work in many cases, but not in all. The dialog now resizes on input, but only seems to extends for one additional row of text. See screenshot (taken on Ubuntu).
Let me know if you think that could be improved without too much effort, and I might take another look.