This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 80533 - Invalid UI layout for customizers
Summary: Invalid UI layout for customizers
Status: VERIFIED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: Schema Tools (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: bhate
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-19 03:08 UTC by Todd Fast
Modified: 2006-09-19 19:58 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
element customizer (22.88 KB, image/jpeg)
2006-08-08 20:04 UTC, bhate
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Todd Fast 2006-07-19 03:08:34 UTC
The default choice for the Type section is "No type", which is the least
desirable choice. Either "Inline complex type" or "Use an existing type are much
more prevalent in real-world use. The order of the radio button choices should
possibly change to reflect user priority.
Comment 1 Jiri Kopsa 2006-07-20 22:24:57 UTC
Let me add a couple of other comments to this dialog:
* Titled borders should not be used.
* It's not clear that the tree chooser below the radio buttons belongs to the
"Use an existing type" radio button. Maybe this radio button should be the last
one in the row.
* There should be a validation message if Ok button is disabled.
* Also when this dialog is in the editing mode, it's not clear what is the
selected type (ok, it's in preview - but that's preview, which may be
overlooked; the information should be in the Swing UI). At least, the visibility
of the selected type should be ensured (suitable scrolling). Better, the radio
button could be accompanied with a text field that would contain the qname of
the selected type.
Comment 2 bhate 2006-07-20 22:57:26 UTC
All customizers are modified.
no titled borders.
alignment is fixed. (Jirka, Todd) pl go through the customizers and let me know
all the issues, if any.
grouping of radio buttons and their corresponding type panels is rectified.
below the type panel we will show current selection with its name, type and
brief description (node description which is any annotation attached to that
component)

what do we want to show  when
* There should be a validation message if Ok button is disabled.
where? at the bottom like import/include customizers?
Comment 3 Todd Fast 2006-07-23 02:38:16 UTC
I don't necessarily agree that there must be a validation message for disabled
OK. It's a nice-to-have, but if it's there, it needs to be consistent with look
and feel in the rest of the IDE (wizards are the only places with these), but
there is no framework nor API for that. I'd rather just leave it out, frankly,
since I don't think it's all that helpful in this case and just adds more
opportunity for inconsistency and maintenance.
Comment 4 bhate 2006-08-07 18:22:23 UTC
When a local element or global element is created, we will show customizer with
inline complex type selected.
It is most commonly used option.
Comment 5 Todd Fast 2006-08-08 06:04:28 UTC
Ajit, can you please attach a screenshot so we can more easily comment?
Comment 6 bhate 2006-08-08 20:04:02 UTC
Created attachment 32664 [details]
element customizer
Comment 7 Todd Fast 2006-08-09 03:52:36 UTC
This looks much better. However, it's not consistent with the NB layout guidelines:

http://xdesign-tools.czech/internal/files/ui-training-presentation-May-Monrovia.odp
Comment 8 bhate 2006-08-16 01:40:14 UTC
Todd, I have verified that the spacing between related components is 6. (exampl
components use existing type and type panel) and spacing between unrelated
components is 11 (name text field and type label for radio buttons).
can you pl. verify?
let me know any issues. (pl point them out)
Comment 9 Jiri Kopsa 2006-08-16 17:34:15 UTC
There is also an issue in the customizer with the validation message. Space for
validation message should be reserved and once it appears, the other components
should stay in their original position and sizes - which is not the case now.
Comment 10 htt 2006-09-19 19:58:34 UTC
As prompted by tfast & jkopsa, marking this as VERIFIED.