+++ This bug was initially created as a clone of Bug #736860 +++ Unify the character used for minus signs on buttons to remove filter rules and filter actions in the Filter editor. I suggest to use \u2212 character as that one is rendered longer and in line with the plus sign. It is already used on the remove filter rule button.
Sorry, the description got copied from the previous bug and I forgot to update it. THE REAL DESCRIPTION IS THIS: Add tooltips and accesskeys to the "add/remove action" button in filter editor. The plus and minus buttons located in the actions list do not have any. The buttons in the rules list have tooltips for what they mean. Any good proposals for the accesskeys?
Created attachment 615027 [details] [diff] [review] patch I am not sure it is wanted to add accesskeys here. See the patch in action. The + sign is underlined on the buttons and minus is in bracket, which is ugly. As the buttons occur several times in the window, the accesskeys work but cycle the focus over all the same buttons. The tooltips should be OK.
Comment on attachment 615027 [details] [diff] [review] patch I agree that we shouldn't have the accesskeys, since they look _really_ odd, but I'm looping in Accessibility Guru MarcoZ to get some feedback on how we should handle access keys for "+" and "-". (Perhaps we should add "(+)" and "(-)" to the end of the tooltiptext?) ui-r=me with that change. Thanks, Blake.
Comment on attachment 615027 [details] [diff] [review] patch Yes, this is good, it will add the tooltip as a description and give the users more context. And yes, having + and - as access keys is odd.
Thanks, so what is the resolution? No accesskeys? And the tooltips as they are now?
aceman: "ui-r=me with that change." ;) So, definitely no accesskeys, and add "(+)" and "(-)" if you think it looks better. (I trust you to make that decision. :) Later, Blake.
Why should I add "(+)" and "(-)" to the tooltip when + and - will not be the accesskeys? I do not understand this part.
D'oh! Yes, it would be better not to add them, then.
Created attachment 617072 [details] [diff] [review] patch v2
Comment on attachment 617072 [details] [diff] [review] patch v2 >+<!ENTITY addAction.label "Add a new action"> >+<!ENTITY removeAction.label "Remove this action"> Nit: tooltiptext entities tend to be foo.tooltip rather than foo.label r=me with that fixed
Comment on attachment 617072 [details] [diff] [review] patch v2 Ian's right about the tooltip change to the string keys. Other than that, this looks pretty solid. Thanks aceman - keep up the good work!
Created attachment 618031 [details] [diff] [review] patch v3 Thanks, fixed.