From 41f7ccd6ac88ff81fce207c470fb23629c158c3e Mon Sep 17 00:00:00 2001 From: ohfp <1813007-ohfp@users.noreply.gitlab.com> Date: Mon, 8 Jun 2020 23:13:03 +0200 Subject: fix broken megabar-removal / urlbar --- megabar.patch | 138 +++++++++++++++++++++++++++++++++++++++++++--------------- 1 file changed, 102 insertions(+), 36 deletions(-) (limited to 'megabar.patch') diff --git a/megabar.patch b/megabar.patch index 02e7f0f..3b03191 100644 --- a/megabar.patch +++ b/megabar.patch @@ -1,15 +1,8 @@ diff --git a/browser/components/urlbar/UrlbarInput.jsm b/browser/components/urlbar/UrlbarInput.jsm -index c557a10..3f8f1d4 100644 +index c557a10..e2c20af 100644 --- a/browser/components/urlbar/UrlbarInput.jsm +++ b/browser/components/urlbar/UrlbarInput.jsm -@@ -1085,18 +1085,18 @@ class UrlbarInput { - } - this.removeAttribute("breakout-extend-disabled"); - -- this._toolbar.setAttribute("urlbar-exceeds-toolbar-bounds", "true"); -- this.setAttribute("breakout-extend", "true"); -+ // this._toolbar.setAttribute("urlbar-exceeds-toolbar-bounds", "true"); -+ // this.setAttribute("breakout-extend", "true"); +@@ -1090,13 +1090,13 @@ class UrlbarInput { // Enable the animation only after the first extend call to ensure it // doesn't run when opening a new window. @@ -30,30 +23,103 @@ index c557a10..3f8f1d4 100644 } endLayoutExtend() { -@@ -1186,8 +1186,13 @@ class UrlbarInput { - let updateKey = {}; - this._layoutBreakoutUpdateKey = updateKey; - -- this.removeAttribute("breakout"); -- this.textbox.parentNode.removeAttribute("breakout"); -+ if (this.hasAttribute("breakout")) { -+ this.removeAttribute("breakout"); -+ } -+ -+ if (this.textbox.parentNode.hasAttribute("breakout")) { -+ this.textbox.parentNode.removeAttribute("breakout"); -+ } - - await this.window.promiseDocumentFlushed(() => {}); - await new Promise(resolve => { -@@ -1209,8 +1214,8 @@ class UrlbarInput { - px(getBoundsWithoutFlushing(this._toolbar).height) - ); - -- this.setAttribute("breakout", "true"); -- this.textbox.parentNode.setAttribute("breakout", "true"); -+ // this.setAttribute("breakout", "true"); -+ // this.textbox.parentNode.setAttribute("breakout", "true"); - - resolve(); - }); +diff --git a/browser/themes/shared/urlbar-autocomplete.inc.css b/browser/themes/shared/urlbar-autocomplete.inc.css +index 13a4595..eda7bdf 100644 +--- a/browser/themes/shared/urlbar-autocomplete.inc.css ++++ b/browser/themes/shared/urlbar-autocomplete.inc.css +@@ -48,8 +48,8 @@ + display: block; + text-shadow: none; + overflow: -moz-hidden-unscrollable; +- margin-inline: @urlbarViewMarginInline@; +- width: calc(100% - 2 * @urlbarViewMarginInline@); ++ margin-inline: 0; ++ width: 100%; + + /* Match urlbar-background's border. */ + border-inline: 1px solid transparent; +@@ -72,7 +72,7 @@ + .urlbarView-row { + fill: currentColor; + fill-opacity: .6; +- padding-block: 3px; ++ padding-block: 0; + } + + .urlbarView-row-inner { +@@ -179,6 +179,7 @@ + background-repeat: no-repeat; + background-size: contain; + -moz-context-properties: fill, fill-opacity; ++ margin-inline-start: 20px; + } + + @media (min-resolution: 2dppx) { +@@ -200,10 +201,10 @@ + + .urlbarView-type-icon { + position: absolute; +- min-width: 12px; +- height: 12px; +- margin-bottom: -4px; +- margin-inline-start: 8px; ++ min-width: 16px; ++ height: 16px; ++ margin-bottom: 0; ++ margin-inline-start: 0; + background-repeat: no-repeat; + background-size: contain; + -moz-context-properties: fill, stroke; +diff --git a/browser/themes/shared/urlbar-searchbar.inc.css b/browser/themes/shared/urlbar-searchbar.inc.css +index 5d5a3d1..4e84a7f 100644 +--- a/browser/themes/shared/urlbar-searchbar.inc.css ++++ b/browser/themes/shared/urlbar-searchbar.inc.css +@@ -6,7 +6,7 @@ + + %define fieldBorderColor hsla(240,5%,5%,.25) + %define fieldHoverBorderColor hsla(240,5%,5%,.35) +-%define urlbarMarginInline 5px ++%define urlbarMarginInline 0px + %define urlbarSearchButtonWidth calc(16px + 2 * @identityBoxPaddingInline@) + + :root { +@@ -196,13 +196,13 @@ + } + + #urlbar[breakout][breakout-extend] { +- top: -@urlbarBreakoutExtend@; +- left: calc(-@urlbarMarginInline@ - @urlbarBreakoutExtend@); +- width: calc(100% + 2 * @urlbarMarginInline@ + 2 * @urlbarBreakoutExtend@); ++ top: calc((var(--urlbar-toolbar-height) - var(--urlbar-height)) / 2); ++ left: 0; ++ width: 100%; + } + + #urlbar[breakout][breakout-extend] > #urlbar-background { +- box-shadow: 0 3px 8px 0 rgba(0,0,0,.15) ++ box-shadow: 0 1px 4px rgba(0,0,0,.05); + } + + #urlbar[breakout][breakout-extend][open] > #urlbar-background { +@@ -210,9 +210,9 @@ + } + + #urlbar[breakout][breakout-extend] > #urlbar-input-container { +- height: calc(var(--urlbar-toolbar-height) + 2 * @urlbarBreakoutExtend@); +- padding-block: calc(@urlbarBreakoutExtend@ + (var(--urlbar-toolbar-height) - var(--urlbar-height)) / 2); +- padding-inline: calc(@urlbarMarginInline@ + @urlbarBreakoutExtend@); ++ height: var(--urlbar-height); ++ padding-block: 0; ++ padding-inline: 0; + } + + #urlbar.searchButton[breakout][breakout-extend] > #urlbar-input-container > #urlbar-search-button { +@@ -230,7 +230,7 @@ + } + + #urlbar[breakout][breakout-extend] > #urlbar-background { +- animation-name: urlbar-grow; ++ animation-name: none; + animation-duration: 0s; + animation-timing-function: var(--animation-easing-function); + } -- cgit