29

On the Close Votes queue, there is Why should this question be closed? indicating what the dialog's purpose is.

On the Suggested Edits, there is nothing.

However, looking at the source, it seems they initially had the title, but the html is messed.

<h2&gt;why are="" you="" rejecting="" this="" edit?<="" h2="">

Edit:

The fix looks great! However...

The new dialog has some padding that looks a bit weird:

The other popups do not have the spacing at the top.

On closer inspection of the source, I see popup-tab-content.

It seems to be causing the spacing, as when I un-nested the form from the div, the result was:

Lastly, the dialog for every other review queue allows movement from the title. (Hover your mouse over, and you can move the dialog)

This is with the code:

<h2 class="popup-title-container handle" style="cursor: move;">
    <span class="popup-breadcrumbs"></span><span class="popup-title">Why are you rejecting this edit?</span>
</h2>

Sorry if I'm being picky :)

Improve this question
2
  • 5
    Localization, it's always localization. :/
    – Shadow Wizard
    Commented Aug 14, 2013 at 19:24
  • +1 because the messed up HTML made me laugh.
    – Donald Duck
    Commented Nov 27, 2017 at 21:48

1 Answer 1

Reset to default
11

Dumb error, literally putting > into a string. Even without localization involvement it would have broken, as Razor will encode any string.

This was fixed in the latest deploy.

Improve this answer
2
  • I have edited my post to reflect the change. sorry if I'm asking too much
    – Dave Chen
    Commented Aug 14, 2013 at 21:37
  • 3
    @DaveChen removal of extra padding and ability to be moved will be in the next build - thanks for pointing it out.
    – Jarrod Dixon
    Commented Aug 15, 2013 at 0:03

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .