GNOME Bugzilla – Bug 98914
[ui-review] "Per Panel Properties" UI issues
Last modified: 2004-12-22 21:47:04 UTC
UI review: Lots of issues with this dialog: - Keyboard navigation is broken with 'Position' (hmmm, works for me) - 'Background' is redundant in the 'type' and 'color' labels - Frames should follow new HIG borderless/bold label style - Would be nice to follow the desktop preferences background dialog -- Tile, Center, Scale, Stretch --- Tile is better than Wallpaper? - Seth disagrees - Background preferences should be verbalized - s/Rotate image for vertical panels/Rotate image when panel is vertical - Remove pixel sizes from dropdown -- panel resizes to fit font now ie. no guarantee panel will stay that size -- instant apply now anyway - Position stuff for floating panels -- Orientation:\n (o) Horizontal\n( )Vertical -- Position:\n Horizontal: [spinner]\nVertical: [spinner] - Corner panel lets you change offset to 0 - if you have a menu panel, no effect until you get to 16 :/ - Corner panel Position buttons are confusing - s/Screen edge offset/Position from edge
- 'Background' is redundant in the 'type' and 'color' labels These labels were agreed in a previous bug (http://bugzilla.gnome.org/show_bug.cgi?id=85628)
- s/Screen edge offset/Position from edge I think "Distance from edge" is better.
They _are_ redundant though. usability-maint: looking at the current dialog, would you recommend keeping "Background type:" etc. on the background tab or changing to simply "Type:" etc?
assign->self I'll look at these issues shortly. usability-maint, can you give some input on the redundancy of "Background" in the 'type' and 'color' labels.
Re: Tile/Wallpaper The AHD defines wallpaper as "A picture or design covering the background of a display screen." The definition of tile that I will shortly add to the Recommended Terminology is "To cover an area with objects that do not overlap." I agree that Tile is the correct term to use on the Background preferences tool. Wallpaper is really just a type of image that is specifically designed for a desktop background. The options on the tool are actions that are performed on the selected image. An argument may be made for wallpaper as a verb, but I think it would still be incorrect - the options on the tool are actions that are performed on the selected image, not on the desktop background.
Well, here's my opinion on the 'Background' issue :) In the bug Eugene refers to, I think the context makes it slightly more important to add the word 'background'... also in that case it was added in direct response to a user request who complained it was too hard to document without it :) Here the two controls appear directly below the word "Background" (on the tab), so I don't think it's quite so necessary to add it here.
> - Keyboard navigation is broken with 'Position' (hmmm, works for me) Works for me too. > - 'Background' is redundant in the 'type' and 'color' labels Done. All arguments taken into account - might as well give it a try. > - Frames should follow new HIG borderless/bold label style Done > - Would be nice to follow the desktop preferences background dialog > -- Tile, Center, Scale, Stretch > --- Tile is better than Wallpaper? - Seth disagrees Done. Changed to T_ile, _Scale, St_retch > - Background preferences should be verbalized Fixed by above I think > - s/Rotate image for vertical panels/Rotate image when panel is > vertical done. > - Remove pixel sizes from dropdown > -- panel resizes to fit font now ie. no guarantee panel will stay > that size > -- instant apply now anyway done. > - Position stuff for floating panels > -- Orientation:\n (o) Horizontal\n( )Vertical > -- Position:\n Horizontal: [spinner]\nVertical: [spinner] Changed, but it still looks shite. > - Corner panel lets you change offset to 0 - if you have a menu > panel, no effect until you get to 16 :/ You mean sliding panel right? Not an easy fix and I don't think its worth the effort. Pretty obvious to the user what is going on. > - Corner panel Position buttons are confusing Don't see why - and I need a suggestion to implement. > - s/Screen edge offset/Position from edge I've used Eugene's suggestion of "Distance from edge" 2002-12-04 Mark McLoughlin <mark@skynet.ie> Fix ui-review issues raised in #98914. * panel-config.c: (make_misc_widget): use "new HIG borderless/bold label style" frame. (floating_notebook_page): ditto and change strings to be a little less confusing. Still a horrible dialog though. (sliding_notebook_page): s/Screen edge offset/Distance from edge/ (make_size_widget): remove pixel sizes from the labels. (background_page): use "new HIG borderless/bold label style" frame. "Verbalise" the image transformation options. Remove redundant "Background" strings. s/Rotate image for vertical panels/ Rotate image when panel is vertical/