summaryrefslogtreecommitdiffstats
path: root/toolkit/content
diff options
context:
space:
mode:
authorNew Tobin Paradigm <email@mattatobin.com>2018-04-13 11:22:17 -0400
committerGitHub <noreply@github.com>2018-04-13 11:22:17 -0400
commitc17c14ce6afc026e8b918b8bf48fba7515bebac3 (patch)
tree61f0bebf9a16479296d6cc9621c86ec23efa348e /toolkit/content
parentaedeefe89855b36da07a021a772cd02c37d6b887 (diff)
parent15e4dcdedd91421e10b80a83b63899439956911a (diff)
downloadUXP-c17c14ce6afc026e8b918b8bf48fba7515bebac3.tar
UXP-c17c14ce6afc026e8b918b8bf48fba7515bebac3.tar.gz
UXP-c17c14ce6afc026e8b918b8bf48fba7515bebac3.tar.lz
UXP-c17c14ce6afc026e8b918b8bf48fba7515bebac3.tar.xz
UXP-c17c14ce6afc026e8b918b8bf48fba7515bebac3.zip
Merge pull request #129 from janekptacijarabaci/_native_52ESR_xulWidgets_menulist_editable_1
Editable menulist loosing editable attribute gets focus stuck (menulist.xml)
Diffstat (limited to 'toolkit/content')
-rw-r--r--toolkit/content/widgets/menulist.xml18
1 files changed, 16 insertions, 2 deletions
diff --git a/toolkit/content/widgets/menulist.xml b/toolkit/content/widgets/menulist.xml
index ccdf3bd26..96c011809 100644
--- a/toolkit/content/widgets/menulist.xml
+++ b/toolkit/content/widgets/menulist.xml
@@ -129,8 +129,22 @@
<property name="label" readonly="true" onget="return this.getAttribute('label');"/>
<property name="description" onset="this.setAttribute('description',val); return val;"
onget="return this.getAttribute('description');"/>
- <property name="editable" onset="this.setAttribute('editable',val); return val;"
- onget="return this.getAttribute('editable') == 'true';"/>
+
+ <property name="editable" onget="return this.getAttribute('editable') == 'true';">
+ <setter>
+ <![CDATA[
+ if (!val && this.editable) {
+ // If we were focused and transition from editable to not editable,
+ // focus the parent menulist so that the focus does not get stuck.
+ if (this.inputField == document.activeElement)
+ window.setTimeout(() => this.focus(), 0);
+ }
+
+ this.setAttribute("editable", val);
+ return val;
+ ]]>
+ </setter>
+ </property>
<property name="open" onset="this.menuBoxObject.openMenu(val);
return val;"