After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 98914 - [ui-review] "Per Panel Properties" UI issues
[ui-review] "Per Panel Properties" UI issues
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: panel
2.1.x
Other other
: Normal minor
: ---
Assigned To: Andrew Sobala
Panel Maintainers
Depends on:
Blocks:
 
 
Reported: 2002-11-18 18:57 UTC by Andrew Sobala
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.1/2.2



Description Andrew Sobala 2002-11-18 18:57:01 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
Comment 1 Eugene O'Connor 2002-11-19 19:21:27 UTC
- '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)



Comment 2 Eugene O'Connor 2002-11-19 19:29:00 UTC
- s/Screen edge offset/Position from edge

I think "Distance from edge" is better.
Comment 3 Andrew Sobala 2002-11-19 19:32:02 UTC
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?
Comment 4 Andrew Sobala 2002-11-23 12:58:00 UTC
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.
Comment 5 Eugene O'Connor 2002-11-24 16:21:33 UTC
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.
Comment 6 Calum Benson 2002-11-27 17:29:32 UTC
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.
Comment 7 Mark McLoughlin 2002-12-04 04:30:24 UTC
> - 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/