GNOME Bugzilla – Bug 85623
[doc] Update docs for per-panel properties dialog
Last modified: 2015-03-24 13:00:31 UTC
The following issues came out of the recent ui-review and are recommended for fixing prior to GNOME 2.0.1: - s/Panel properties/Panel Properties on the window title - s/panel/Panel (on the tabs) - Dialog needs mnemonics. - Set default button to Close - Remove the note that says 'The panel will size itself to the largest applet ...'- This should be covered in Help only - s/Enable Auto-hide/Autohide - s/Panel Position/Panel position: - s/Show arrows on hide buttons/Arrows on hide buttons - s/Standard/Default in drop down list - s/Pixmap/Image in dropdown list - s/Background Type/Background type: - s/Don't scale image to fit/Do not scale image to fit - Remove the text in parentheses
Adding keywords to all ui-review bugs in one mass change, sorry for the spam. It would be very nice if the folks doing UI review stuff would be sure to add 'PATCH' to the bugs that have those. I'm (tentatively) not marking these 'high' but the release team [having gotten approval from the docs and i18n people] would like to see these in ASAP on the 2.0.1 branches, so that i18n folks can start translating at their convenience.
Calum, I had a discussion about this dialog the last time, and that removing this stuff, makes it look arsey. The patch again is in 82586.
Looking arsey can always be fixed with some glade-love, it's behaving arsey I'm concerned about :)
After¦some¦discussion¦with¦the¦release¦team,¦in¦order¦to¦get¦these¦in¦to¦2.0.1 and¦to¦not¦make¦the¦translators¦shoot¦us,¦these¦need¦to¦get¦in¦ASAP¦:)¦I'm marking¦them¦'urgent'/2.0.1¦as¦a¦result;¦it¦would¦be¦greatly¦appreciated¦if maintainers¦of¦the¦affected¦components¦can¦work¦on¦getting¦these¦patched¦ASAP¦so that¦the¦translators¦can¦get¦cracking¦on¦them¦for¦the¦next¦release.¦[Subject,¦of course,¦to¦maintainer¦agreement¦with¦the¦goals¦of¦the¦review¦:)¦
If only it used glade...[sigh]
I've got a patch for this...but having serious spacing issues - all the rest of the stuff is done though.
Glynn, *you* may have a patch - but I don't have it :-) Implemented Calum's changes on HEAD.
This bug should not be marked as resolve. The following items have not been resolved. - Dialog needs mnemonics. - Set default button to Close
The default response is set to close, but quite right, I didn't do mnemonics. Could someone suggest some sensible ones. I can't even spell the word, so I don't trust myself coming up with decent ones :-) Bear in mind that the dialog is different for the different types of panels.
Does it look like arse now...huh huh huh? :) I'm still working on the patch I have here....it's not for the public quite yet. Mine looks like arse too.
Created attachment 9593 [details] [review] ui review for panel config dialogs
Okay, I've attached the UI / String changes....there's some points I'd like to made - - UI is reasonably arse...but without this patch it looks more arse - I'm getting a weird mnemonic problem on Sc_ale Image where it doesn't select the radio button [no clue why that is happening] - _Position and mnemonic needs some thought and probably should update it's mnemonic to which ever button is depressed at the time
s/Autohide/Hide panel when unused/ <- Jargon adds barriers to understanding, even if it sounds cool. s/Image in drop-down list/Image/ <- At the point that the user selects this item, "drop-down list" is a confusing piece of information that won't mean anything to them and may scare them away from the item. Other than that, looks good.
Agree with the 'Hide panel when unused'...i think that's a good addition.. With regard to the other, i think you must be smoking some pretty strong stuff there Seth...not quite sure where you're getting 'Image in drop-down list' from :/
"- s/Pixmap/Image in dropdown list" Augh. I read this wrong ;-) I read this as: s/Pixmap/Image in dropdown list/ rather than: s/Pixmap/Image/ (in the dropdown list) silly me.
Let us not forget that once these changes are done, the docs need to be redone. Please don't close until that is completed. Adding Eugene to cc list, as these are in the Users Guide.
Glynn, I've committed your patch with a couple of modifications 1) Removed the extraneous vboxes 2) Put the 'misc' widget at the end (talk about arse ui :-) Re-assigning to help-docs component
the extra vboxes were probably there to provide the proper spacing. did you have a look at what the misc widget is? yeah, it's screen placement or 'position' ;) that's why i put the other stuff at the bottom...also for space reasons too. I still need to fix up the mnemonics on the 'Position:' field they don't work at all currently.
Oh, probably should remove the frame too.
Created attachment 9735 [details] [review] removing the frame
or even a patch that compiles...ugh. s/return vbox/return box [sigh]
Forget the Misc section. It will be re-worked properly with multihead. Please stop re-opening this bug. I consider the ui review changes implemented. If there is another ui review, with new changes, they should go into a new bug.
Trimming keywords, changing priority and updating title for clarity.
Guys, whats the status on this - have the docs been updated ?
The docs have been updated to reflect these changes.