Commit a654e2a5 authored by binji@chromium.org's avatar binji@chromium.org

[Web UI Options] For overlays that have >1 button-strip, reverse order of all (in views).

BUG=102303
TEST=Run views chrome. Go to chrome://settings/personal. Sign in to sync. Click "Advanced". OK button should be on left. In non-views chrome (mac, linux), OK button should be on right.


Review URL: http://codereview.chromium.org/8387033

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108014 0039d316-1c4b-4281-b951-d872f2087c98
parent c020ea3d
...@@ -481,15 +481,16 @@ cr.define('options', function() { ...@@ -481,15 +481,16 @@ cr.define('options', function() {
* @private * @private
*/ */
OptionsPage.reverseButtonStrip_ = function(overlay) { OptionsPage.reverseButtonStrip_ = function(overlay) {
var buttonStrip = overlay.pageDiv.querySelector('.button-strip'); var buttonStrips = overlay.pageDiv.querySelectorAll('.button-strip');
// Not all overlays have button strips. // Reverse all button-strips in the overlay.
if (!buttonStrip) for (var j = 0; j < buttonStrips.length; j++) {
return; var buttonStrip = buttonStrips[j];
var childNodes = buttonStrip.childNodes; var childNodes = buttonStrip.childNodes;
for (var i = childNodes.length - 1; i >= 0; i--) for (var i = childNodes.length - 1; i >= 0; i--)
buttonStrip.appendChild(childNodes[i]); buttonStrip.appendChild(childNodes[i]);
}
}; };
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment