Last modified: 2010-05-15 15:37:22 UTC
With the addition of the "Show changes" button, the line of buttons now looks a bit confusing. (Will upload screenshot) Possible solutions include bolding the "Save page" button and adding (for example) an em space (   or   or   ) after the "Save page" button to separate it from the less important buttons.
Created attachment 374 [details] Screenshot of problem as seen from an admin account (1024x768) Mozilla Firefox 1.0.1 on Windows XP. Developer toolbar accidentally left in.
  commited to head. Thanks :o)
Problem still appears on FiverAlpha.
(In reply to comment #2) >   commited to head. Thanks :o) Ashar, I removed that space a few days ago ( in version 1.165 ), first of all it looks like crap in some browsers, namely Konqueror and css is the right place to do this, if you really want to add a margin on that button please do so in CSS where it belongs.
Well, will anybody make this change? Here's the addition to main.css (for Monobook): #wpSave { margin-right: 1em; }
Forgot to mention that the exact same thing can be used for any other skin. Users can also put it in their own /Monobook.css file as I will.
(In reply to comment #5) > Here's the addition to main.css (for Monobook): > > #wpSave { > margin-right: 1em; > } Why put a space between the save and preview buttons? It wasn't like that in 1.4. Furthermore, why have a space between them at all? WONTFIX?
Well, I felt that with the addition of "Show pages" the whole thing looked too crammed. It's the logical place to distance the three buttons.
(In reply to comment #8) > Well, I felt that with the addition of "Show pages" the whole thing looked too > crammed. It's the logical place to distance the three buttons. You're right, I put a 1em right margin to #wpSave in HEAD as you suggested, marking this as FIXED.
I prefer the following CSS for monobook/main.css, and it happens to match the IE default. Firefox uses smaller internal and external margins, which is where I see the problem. N.B. The CSS needs to be margin-left in monobook/rtl.css. input#wpSave, input#wpPreview, input#wpDiff { margin-right: 0.33em; }
(In reply to comment #10) > I prefer the following CSS for monobook/main.css, and it happens to match the IE > default. Firefox uses smaller internal and external margins, which is where I > see the problem. N.B. The CSS needs to be margin-left in monobook/rtl.css. > > input#wpSave, input#wpPreview, input#wpDiff { > margin-right: 0.33em; > } > Applied input#wpSave, input#wpDiff { margin-right: 0.33em; } To HEAD, that way the "Show preview" and "Show changes" buttins are grouped togather.
(In reply to comment #11) > (In reply to comment #10) I was wrong about IE, due to an IE6 CSS bug during a previous test. Thanks for the prompt commit Ævar, but can you do monobook/rtl.css as well?
(In reply to comment #12) > (In reply to comment #11) > > (In reply to comment #10) > I was wrong about IE, due to an IE6 CSS bug during a previous test. > > Thanks for the prompt commit Ævar, but can you do monobook/rtl.css as well? Can you provide a patch? I find thinking in ltr terms mind boggling.
(In reply to comment #13) This can be added to the bottom of monobook/rtl.css : input#wpSave, input#wpDiff { margin-left: 0.33em; }
(In reply to comment #14) > (In reply to comment #13) > This can be added to the bottom of monobook/rtl.css : Corrected to override main.css: input#wpSave, input#wpDiff { margin-right: 0; margin-left: 0.33em; }
