|
[
Permalink
| « Hide
]
Wen Hao Chuang added a comment - 22/May/08 02:36 AM
OK I'm adding Martin D to the watch list as we discussed few months ago. Martin please let me know what you think, thanks!
Assigning this to Dongsheng... my +1 for it (18_STABLE, 19_STABLE and HEAD). Ask MD for confirmation, please.
Thanks for report and patch Wen! Ciao Quick comments: take out the inline styles (must be in theme), and: would this get big in other languages? Can we not use alt tags or something similar for JAWS?
The inline styles are taken out from the file, put a new patch here.
It seems nobody oppose the new patch, see I committed the improvement. Feel free to reopen. Thanks
Dear Dongsheng, just my $0.02, I think you added the changes in styles_layout.css at the bottom, maybe in the future we could add it to corresponding sections in the styles_layout.css as there are many sections in styles_layout.css? For this change maybe it should go into the admin section? Just my humble opinion...so if I may I'm reopening this one for now so that we could add the
#add, #remove { to the *** Course section (around line #1729), what do people think? Thanks! Wenhao, fixed, please review.
I'm not sure this change is a good idea.
It takes more space on the screen, and let less place for usefull information  Would it be possible to put these as Alt text, as MartinD suggested ? And the text is not entirely shown (see attachment) hmm, I like it - I had problems with the old arrows in "non-standard" browsers
Hi, Séverin, alt text is for image input.
so...
Should I revert to the old version? This fix was mainly to address "accessibility" issues (for screen readers like JAWS), although it might have some pros and cons with "usability". I guess more and more universities will be following similar accessible technology initiative like the CSU one (see: http://www.calstate.edu/accessibility/
 ), which is why I would vote to keep this fix in the core (to reduce trouble for most schools to have to keep this as their local customization). Just my 2 cents.
Séverin, which Web browser with what resolution did you use to generate that cut-off "remove" button for your screenshot? Just curious.. thanks! Yes don't revert, because
1) Different languages can choose appropriate strings there (including "" if they must). 2) The cutoff thing looks like a CSS bug perhaps? It can probably be fixed. Séverin, are you using the old version IE?
Sorry for the late answer, didn't see all comments.
I think i was tired when putting this comment : it's related to group assignment I've just tested latest CVS version, with Firefox 3 (i was using 2.0.0.14 when putting my first comment), and there are no alignment problem in role assignment, and text is all visible But, for group assignment, problem is still there : text isn't all contained (in french), and the arrows are not the same (the two on the left) as in role assignment (one arrow on the left, the other one on the right). Would be good if interface was consistent I attached another patch here, it fixed the button in group assigning page, tested in French, English and Chinese.
Dongsheng, I don't see any changes in that patch (except for white space) ... is something missing?
group assignment screen shot
Sorry, I attached a wrong file
No probs, can you also look at the Bulk user actions screen to see if it's possible to use the same screen there? admin/user/user_bulk.php
Dongsheng - I've added a screen shot of the bulk user action page so you can see what it looks like which is very similar to what you've been doing in fixing up this one. I'll had that this issue has some type of relationship to
If Dongsheng is able to work on the bulk user actions page (/admin/user/user_bulk.php) it will help to improve the consistency of the user interface and thus help address the question of
I removed the space in add/delete button, and align the text in the center of the button, not sure if this is the best practice, because different language has different length.
Hi, Anthony, no problem, I will work on bulk user page.
The buttons are fixed in 1.7, 1.8, 1.9 and HEAD.
works for now, thanks
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Improvement
Closed
Minor
Improve accessibility of 'assign role' button and the consistency of the UI in bulk user actions page
 |
|
All content on this web site is made available under the GNU General Public License, unless otherwise stated.
