Bug 1400219 - Align permission status menulists in the permission dialog with the header and adjust the menulist margin. r=johannh
authorPrathiksha <prathikshaprasadsuman@gmail.com>
Fri, 08 Sep 2017 19:11:02 +0530
changeset 669696 e5c4c1634e3af703e873dbbb1559d9d9ddb4faaa
parent 669695 344a2ba91f955e359ef560aedd778a9cd521a89b
child 669697 b097014498107d5b56744b2946717dae2747cf2d
push id81398
push userhikezoe@mozilla.com
push dateMon, 25 Sep 2017 08:14:31 +0000
reviewersjohannh
bugs1400219
milestone58.0a1
Bug 1400219 - Align permission status menulists in the permission dialog with the header and adjust the menulist margin. r=johannh MozReview-Commit-ID: LaY1YaNZydg
browser/components/preferences/sitePermissions.css
browser/components/preferences/sitePermissions.js
--- a/browser/components/preferences/sitePermissions.css
+++ b/browser/components/preferences/sitePermissions.css
@@ -11,8 +11,13 @@
   min-height: 18em;
 }
 
 #siteCol,
 #statusCol,
 #permissionsBox > richlistitem {
   min-height: 35px;
 }
+
+.website-status {
+  margin: 1px;
+  margin-inline-end: 5px;
+}
--- a/browser/components/preferences/sitePermissions.js
+++ b/browser/components/preferences/sitePermissions.js
@@ -148,16 +148,17 @@ var gSitePermissionsManager = {
     hbox.setAttribute("class", "website-name");
     hbox.setAttribute("flex", "3");
     hbox.appendChild(website);
 
     let menulist = document.createElement("menulist");
     let menupopup = document.createElement("menupopup");
     menulist.setAttribute("flex", "1");
     menulist.setAttribute("width", "50");
+    menulist.setAttribute("class", "website-status");
     menulist.appendChild(menupopup);
     let states = SitePermissions.getAvailableStates(permission.type);
     for (let state of states) {
       if (state == SitePermissions.UNKNOWN)
         continue;
       let m = document.createElement("menuitem");
       m.setAttribute("label", this._getCapabilityString(state));
       m.setAttribute("value", state);