mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-12 12:55:46 +00:00
Fix 30878: right clicking on buttons should not trigger button action, and pressing spacebar needs to trigger all buttons. more to come. r=kin sr=sfraser
This commit is contained in:
parent
7e2782ec3e
commit
689f92be09
@ -436,7 +436,7 @@
|
||||
|
||||
<menuitem id="fontColor" value="&formatFontColor.label;"
|
||||
accesskey="&formatfontcolor.accesskey;"
|
||||
oncommand="EditorSelectColor('Text')"
|
||||
oncommand="EditorSelectColor('Text');"
|
||||
position="4"/>
|
||||
<menuseparator position="5"/>
|
||||
|
||||
@ -699,10 +699,10 @@
|
||||
<observes element="cmd_fontColor" attribute="state" onbroadcast="onFontColorChange()"/>
|
||||
<observes element="cmd_backgroundColor" attribute="state" onbroadcast="onBackgroundColorChange()"/>
|
||||
<text class="color-button" id="BackgroundColorButton"
|
||||
onclick="EditorSelectColor('')"
|
||||
oncommand="EditorSelectColor('');"
|
||||
tooltip="aTooltip" tooltiptext="&BackgroundColorButton.tooltip;"/>
|
||||
<text class="color-button" id="TextColorButton"
|
||||
onclick="EditorSelectColor('Text')"
|
||||
oncommand="EditorSelectColor('Text');"
|
||||
tooltip="aTooltip" tooltiptext="&TextColorButton.tooltip;"/>
|
||||
</stack>
|
||||
|
||||
@ -762,13 +762,13 @@
|
||||
</button>
|
||||
|
||||
<!-- Edit Mode toolbar -->
|
||||
<tab id="NormalModeButton" class="tab-bottom edit-mode _plain" type="text" selected="1" value="&NormalModeTab.label;" onclick="SetEditMode(1)"
|
||||
<tab id="NormalModeButton" class="tab-bottom edit-mode _plain" type="text" selected="1" value="&NormalModeTab.label;" oncommand="SetEditMode(1);"
|
||||
tooltip="aTooltip" tooltiptext="&NormalMode.tooltip;"/>
|
||||
<tab id="TagModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&AllTagsMode.label;" onclick="SetEditMode(2)"
|
||||
<tab id="TagModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&AllTagsMode.label;" oncommand="SetEditMode(2);"
|
||||
tooltip="aTooltip" tooltiptext="&AllTagsMode.tooltip;"/>
|
||||
<tab id="SourceModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&SourceMode.label;" onclick="SetEditMode(3)"
|
||||
<tab id="SourceModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&SourceMode.label;" oncommand="SetEditMode(3);"
|
||||
tooltip="aTooltip" tooltiptext="&HTMLSourceMode.tooltip;"/>
|
||||
<tab id="PreviewModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&PreviewMode.label;" onclick="SetEditMode(0)"
|
||||
<tab id="PreviewModeButton" class="tab-bottom edit-mode _plain" type="text" selected="0" value="&PreviewMode.label;" oncommand="SetEditMode(0);"
|
||||
tooltip="aTooltip" tooltiptext="&PreviewMode.tooltip;"/>
|
||||
|
||||
<!-- InsertPopupButton is used by messengercompose.xul -->
|
||||
|
@ -79,7 +79,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Text" value="&normalText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="TextButton" class="dialog color-button" onclick="GetColorAndUpdate('textCW')"/>
|
||||
<button id="TextButton" class="dialog color-button" oncommand="GetColorAndUpdate('textCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="textCW" class="color-well"/>
|
||||
@ -90,7 +90,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Link" value="&linkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="LinkButton" class="dialog color-button" onclick="GetColorAndUpdate('linkCW')"/>
|
||||
<button id="LinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('linkCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="linkCW" class="color-well"/>
|
||||
@ -101,7 +101,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Active" value="&activeLinkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="ActiveLinkButton" class="dialog color-button" onclick="GetColorAndUpdate('activeCW')"/>
|
||||
<button id="ActiveLinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('activeCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="activeCW" class="color-well"/>
|
||||
@ -112,7 +112,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Visited" value ="&visitedLinkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="VisitedLinkButton" class="dialog color-button" onclick="GetColorAndUpdate('visitedCW')"/>
|
||||
<button id="VisitedLinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('visitedCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="visitedCW" class="color-well"/>
|
||||
@ -123,7 +123,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Background" value="&background.label;" />
|
||||
<stack>
|
||||
<button id="BackgroundButton" class="dialog color-button" onclick="GetColorAndUpdate('backgroundCW')"/>
|
||||
<button id="BackgroundButton" class="dialog color-button" oncommand="GetColorAndUpdate('backgroundCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="backgroundCW" class="color-well"/>
|
||||
@ -153,7 +153,7 @@
|
||||
|
||||
<text class="label" value="&backgroundImage.label;" accesskey="&backgroundImage.accesskey;"/>
|
||||
<box valign="middle" autostretch="never">
|
||||
<textfield id="BackgroundImageInput" onkeyup="ChangeBackgroundImage()" style="min-width : 16em"/>
|
||||
<textfield id="BackgroundImageInput" onkeyup="ChangeBackgroundImage();" style="min-width : 16em"/>
|
||||
<button class="dialog" value="&chooseFile.label;" accesskey="&chooseFile.accesskey;"
|
||||
oncommand="ChooseImageFile()"/>
|
||||
<data id="backgroundImageData" pref="true" preftype="string" prefstring="editor.default_background_image" prefattribute="value"/>
|
||||
|
@ -54,7 +54,7 @@
|
||||
<!-- </radiogroup> -->
|
||||
<colorpicker id="ColorPicker" palettename="standard"
|
||||
persist="palettename"
|
||||
onclick="SelectColor()"/>
|
||||
oncommand="SelectColor();"/>
|
||||
<!-- Web palette is not implemented???
|
||||
<box valign="middle" autostretch="never">
|
||||
<text class="label" value="&palette.label;"/>
|
||||
@ -67,7 +67,7 @@
|
||||
<stack valign="middle" autostretch="never">
|
||||
<button id="LastPickedButton" class="dialog" value="&lastPickedColor.label;"
|
||||
style="margin-left:0px; margin-right: 0px;"
|
||||
onclick="SelectLastPickedColor()"/>
|
||||
oncommand="SelectLastPickedColor();"/>
|
||||
<box style="margin-left: 5px" mousethrough="always">
|
||||
<box orient="vertical" autostretch="never">
|
||||
<spring flex="1"/>
|
||||
|
@ -64,7 +64,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Text" value="&normalText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="TextButton" class="dialog color-button" onclick="GetColorAndUpdate('textCW')"/>
|
||||
<button id="TextButton" class="dialog color-button" oncommand="GetColorAndUpdate('textCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="textCW" class="color-well"/>
|
||||
@ -75,7 +75,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Link" value="&linkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="LinkButton" class="dialog color-button" onclick="GetColorAndUpdate('linkCW')"/>
|
||||
<button id="LinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('linkCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="linkCW" class="color-well"/>
|
||||
@ -86,7 +86,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Active" value="&activeLinkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="ActiveLinkButton" class="dialog color-button" onclick="GetColorAndUpdate('activeCW')"/>
|
||||
<button id="ActiveLinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('activeCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="activeCW" class="color-well"/>
|
||||
@ -97,7 +97,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Visited" value ="&visitedLinkText.label;&colon.character;" />
|
||||
<stack>
|
||||
<button id="VisitedLinkButton" class="dialog color-button" onclick="GetColorAndUpdate('visitedCW')"/>
|
||||
<button id="VisitedLinkButton" class="dialog color-button" oncommand="GetColorAndUpdate('visitedCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="visitedCW" class="color-well"/>
|
||||
@ -108,7 +108,7 @@
|
||||
<row valign="middle" autostretch="never">
|
||||
<text class="label" id="Background" value="&background.label;" />
|
||||
<stack>
|
||||
<button id="BackgroundButton" class="dialog color-button" onclick="GetColorAndUpdate('backgroundCW')"/>
|
||||
<button id="BackgroundButton" class="dialog color-button" oncommand="GetColorAndUpdate('backgroundCW');"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="backgroundCW" class="color-well"/>
|
||||
|
@ -67,7 +67,7 @@
|
||||
<button class="dialog" id="RemoveWord" oncommand="RemoveWord()" value="&RemoveButton.label;"/>
|
||||
<spring class="spacer"/>
|
||||
<spring flex="1"/>
|
||||
<button class="exit-dialog" id="close" value="&CloseButton.label;" default="true" onclick="Close()"/>
|
||||
<button class="exit-dialog" id="close" value="&CloseButton.label;" default="true" oncommand="Close();"/>
|
||||
</box>
|
||||
</row>
|
||||
</rows>
|
||||
|
@ -159,10 +159,10 @@
|
||||
|
||||
<toolbar id="toolbox" orient="vertical">
|
||||
<box orient="vertical">
|
||||
<button id="pointerButton" class="tool" toggled="1" onclick="changeTool(event, 'pointer')"/>
|
||||
<button id="rectButton" class="tool" onclick="changeTool(event, 'rect')"/>
|
||||
<button id="cirButton" class="tool" onclick="changeTool(event, 'cir')"/>
|
||||
<button id="polyButton" class="tool" onclick="changeTool(event, 'poly')"/>
|
||||
<button id="pointerButton" class="tool" toggled="1" oncommand="changeTool(event, 'pointer');"/>
|
||||
<button id="rectButton" class="tool" oncommand="changeTool(event, 'rect');"/>
|
||||
<button id="cirButton" class="tool" oncommand="changeTool(event, 'cir');"/>
|
||||
<button id="polyButton" class="tool" oncommand="changeTool(event, 'poly');"/>
|
||||
<spring flex="1"/>
|
||||
</box>
|
||||
</toolbar>
|
||||
@ -188,7 +188,7 @@
|
||||
class = "push dialog"
|
||||
id = "DelButton"
|
||||
value="&delSpotCmd.label;"
|
||||
onclick="deleteElement(currentElement)"/> -->
|
||||
oncommand="deleteElement(currentElement);"/> -->
|
||||
|
||||
<spring flex="1"/>
|
||||
|
||||
|
@ -127,7 +127,7 @@
|
||||
<box valign="middle" autostretch="never">
|
||||
<text class="label" value="&backgroundColor.label;"/>
|
||||
<stack>
|
||||
<button class="dialog color-button" onclick="GetColorAndUpdate('tableBackgroundCW')" flex="1"/>
|
||||
<button class="dialog color-button" oncommand="GetColorAndUpdate('tableBackgroundCW');" flex="1"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="tableBackgroundCW" class="color-well"/>
|
||||
@ -256,7 +256,7 @@
|
||||
<box valign="middle" autostretch="never">
|
||||
<checkbox id="CellColorCheckbox" value="&backgroundColor.label;"/>
|
||||
<stack>
|
||||
<button class="dialog color-button" onclick="GetColorAndUpdate('cellBackgroundCW')" flex="1"/>
|
||||
<button class="dialog color-button" oncommand="GetColorAndUpdate('cellBackgroundCW');" flex="1"/>
|
||||
<box autostretch="never" valign="middle" mousethrough="always">
|
||||
<spring class="smallspacer"/>
|
||||
<spring id="cellBackgroundCW" class="color-well"/>
|
||||
|
Loading…
Reference in New Issue
Block a user