Bug 42056 - [Dialogs] New Path Variable Dialog; lable alignment
Summary: [Dialogs] New Path Variable Dialog; lable alignment
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.2 RC2   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
Depends on:
Blocks:
 
Reported: 2003-08-26 20:01 EDT by Sebastian Davids CLA
Modified: 2006-04-27 13:03 EDT (History)
3 users (show)

See Also:


Attachments
fix (10.30 KB, patch)
2006-03-17 10:22 EST, Sebastian Davids CLA
no flags Details | Diff
pic showing old (top) and new (bottom) look (20.02 KB, image/png)
2006-03-17 10:23 EST, Sebastian Davids CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sebastian Davids CLA 2003-08-26 20:01:07 EDT
The "Name" and "Location" labels should be bottom aligned with their
corresponding text fields.
Comment 1 Tod Creasey CLA 2003-08-27 09:01:41 EDT
Sebastian do you mean the New Dialog from the Java Classpath variables page?
Comment 2 Dirk Baeumer CLA 2003-08-27 09:08:12 EDT
The whole Eclipse UI never bottom alignes labels with text fields. They are 
always centered. If we change this we should change it for whole Eclipse. 
Asking Platform for comments if there are UI guidelines how to aligne labels 
and text entry fields.
Comment 3 Tod Creasey CLA 2003-08-27 09:22:45 EDT
The way you are doing it is pretty much the defacto standard for Eclipse - we 
generally do not constrain if there is not a strong reason to do so.

I would be inclined to not change your dialog.
Comment 4 Sebastian Davids CLA 2003-08-27 09:25:52 EDT
Preferences Workbench/Linked Resources

New...

@@@@

The "visual" difference between bottom and center alignment is quite small.

The top aligned labels look weird though.

@@@@

I think there should be an UI Guideline.

Either center or bottom alignment would be fine with me.
Comment 5 Dirk Baeumer CLA 2003-08-27 09:28:21 EDT
We should follow the Eclipse defacto standard and should center labels and 
entry fields.

Tod, may be we should add a sentence to the UI guideline paper. 

No further action planned in JDT/UI.
Comment 6 Sebastian Davids CLA 2003-08-27 09:40:27 EDT
A quick investigation revealed several bottom aligned labels.

Should I file bugs for them?

@@@@

I reopen this bug because you've probably looked at the dialog from comment #1.

The dialog I mentioned does belong to Platform/UI.
Comment 7 Dirk Baeumer CLA 2003-08-27 09:45:50 EDT
Regarding filing bugs: I would do so, but Platform/UI should make clear what 
the default should be.

The bug is assigned to JDT/UI this is why I closed it. Moving to platform then.
Comment 8 Tod Creasey CLA 2003-08-27 09:51:51 EDT
The inconsistency is due to us never having made a decision one way or the 
other about this.

I will rename the problem report and assign it to Nick who is looking into 
guidelines now. If you want to point out anything to us please add to this PR 
rather than creating a pile more.
Comment 9 Sebastian Davids CLA 2003-09-01 08:01:24 EDT
Regardless of the UI Guidelines outcome; the "top" alignment of the labels in
this dialog should be changed to "center" alignment.
Comment 10 Nick Edgar CLA 2006-03-15 13:45:34 EST
Reassigning bugs in component areas that are changing ownership.
Comment 11 Sebastian Davids CLA 2006-03-17 10:22:26 EST
Created attachment 36491 [details]
fix

- dialog resizable now
- updated copyright
- replaced unneccessary setFont calls w/ Dialog.applyDialogFont
- standard margins at bottom/right/top/left and between widgets
- proper resize behavior
- labels center aligned
Comment 12 Sebastian Davids CLA 2006-03-17 10:23:26 EST
Created attachment 36492 [details]
pic showing old (top) and new (bottom) look
Comment 13 Susan McCourt CLA 2006-03-17 13:05:41 EST
marking for 3.2 so as not to lose track of this one
Comment 14 Susan McCourt CLA 2006-04-17 19:03:26 EDT
Fixed >20060417.
Sebastian, the only thing I didn't like about the patch was the change in tab order caused by moving the creation of the variable name and value fields to the bottom of the method.  Instead, I maintained the creation order from before and moved the layout code to the bottom.

This also allowed me to change the label creation order so that the labels are created just before their corresponding fields (more screen-reader friendly).
Comment 15 Sebastian Davids CLA 2006-04-18 07:19:58 EDT
Good point, forgot about the Tab-order :|
Comment 16 Susan McCourt CLA 2006-04-21 14:50:54 EDT
cc'ing McQ for FYI.
Comment 17 Susan McCourt CLA 2006-04-27 13:03:17 EDT
verified on WinXP, N20060425-0010