Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/options.xul

Issue 8478020: Slight improvements to URL Fixer code (Closed)
Patch Set: Created Oct. 1, 2012, 4:03 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/content/options.js ('k') | lib/typoFixer.js » ('j') | lib/typoFixer.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/options.xul
===================================================================
--- a/chrome/content/options.xul
+++ b/chrome/content/options.xul
@@ -36,18 +36,18 @@
</listcols>
</listbox>
<hbox>
<button flex="1" label="&urlfixer.options.removeButtonLabel;" onclick="removeRule(this,'whitelist');" disabled="true" id="whitelist_remove-button" _list="whitelist" />
</hbox>
</tabpanel>
<tabpanel orient="vertical">
<hbox>
- <textbox id="find" flex="1" oninput="onFindChange();" onkeypress="if (event.keyCode == 13) { addRule();event.preventDefault(); }" />
- <textbox id="replace" flex="1" onkeypress="if (event.keyCode == 13) { addRule();event.preventDefault(); }" />
+ <textbox id="find" flex="1" oninput="onFindChange();" onkeypress="if (event.keyCode == event.DOM_VK_RETURN) { addRule();event.preventDefault(); }" />
+ <textbox id="replace" flex="1" onkeypress="if (event.keyCode == event.DOM_VK_RETURN) { addRule();event.preventDefault(); }" />
</hbox>
<hbox>
<button flex="1" label="&urlfixer.options.addButtonLabel;" oncommand="addRule();" id="add-button" disabled="true" />
</hbox>
<description id="custom_corrections-explanation">&urlfixer.options.regularExpressionExplanation;</description>
<listbox id="custom_corrections" flex="1" seltype="multiple" onselect="onItemSelected(this);" _removeButton="custom_corrections_remove-button" _emptyLabel="&urlfixer.options.findAndReplaceEmptyLabel;">
<listhead>
<listheader label="&urlfixer.options.findLabel;"/>
« no previous file with comments | « chrome/content/options.js ('k') | lib/typoFixer.js » ('j') | lib/typoFixer.js » ('J')

Powered by Google App Engine
This is Rietveld