superreview requested: [Bug 313027] Can't drag plain text url to tab bar : [Attachment 205462] Patch to make url parsing from text less strict

Stuart Morgan <stuart.morgan@alumni.case.edu> has asked  for superreview:
Bug 313027: Can't drag plain text url to tab bar
https://bugzilla.mozilla.org/show_bug.cgi?id=313027

Attachment 205462: Patch to make url parsing from text less strict
https://bugzilla.mozilla.org/attachment.cgi?id=205462&action=edit

------- Additional Comments from Stuart Morgan <stuart.morgan@alumni.case.edu>
This seems like the way to go, and works in my testing.

r=me (although the commented out |&& ...| should be removed before checkin.
0
bugzilla
7/12/2006 4:29:00 AM
mozilla.dev.super-review 29307 articles. 3 followers. Post Follow

0 Replies
632 Views

Similar Articles

[PageSpeed] 55

Reply:

Similar Artilces:

superreview requested: [Bug 313027] Can't drag plain text url to tab bar : [Attachment 205462] Patch to make url parsing from text less strict #2
Stuart Morgan <stuart.morgan@alumni.case.edu> has asked Mike Pinkerton <mikepinkerton@mac.com> for superreview: Bug 313027: Can't drag plain text url to tab bar https://bugzilla.mozilla.org/show_bug.cgi?id=313027 Attachment 205462: Patch to make url parsing from text less strict https://bugzilla.mozilla.org/attachment.cgi?id=205462&action=edit ...

superreview granted: [Bug 313027] Can't drag plain text url to tab bar : [Attachment 205462] Patch to make url parsing from text less strict
Mike Pinkerton <mikepinkerton@mac.com> has granted Stuart Morgan <stuart.morgan@alumni.case.edu>'s request for superreview: Bug 313027: Can't drag plain text url to tab bar https://bugzilla.mozilla.org/show_bug.cgi?id=313027 Attachment 205462: Patch to make url parsing from text less strict https://bugzilla.mozilla.org/attachment.cgi?id=205462&action=edit ------- Additional Comments from Mike Pinkerton <mikepinkerton@mac.com> sr=pink yes, please remove the commented out bits of the expression. ...

superreview requested: [Bug 162791] PATCH: URL bar accepts Drag & Drop of strings/URLs : [Attachment 122076] doesn't load URL on drop
Simon Woodside <sbwoodside@yahoo.com> has asked Mike Pinkerton <pinkerton@aol.net> for superreview: Bug 162791: PATCH: URL bar accepts Drag & Drop of strings/URLs http://bugzilla.mozilla.org/show_bug.cgi?id=162791 Attachment 122076: doesn't load URL on drop http://bugzilla.mozilla.org/attachment.cgi?id=122076&action=edit ------- Additional Comments from Simon Woodside <sbwoodside@yahoo.com> r+ ...

superreview requested: [Bug 78786] [FIX] Can't scroll URL bar history drop down with mouse wheel : [Attachment 153557] Patch A rev. 1 (must have patch from bug 97283 first)
Mats Palmgren <mats.palmgren@bredband.net> has asked David Baron <dbaron@dbaron.org> for superreview: Bug 78786: [FIX] Can't scroll URL bar history drop down with mouse wheel http://bugzilla.mozilla.org/show_bug.cgi?id=78786 Attachment 153557: Patch A rev. 1 (must have patch from bug 97283 first) http://bugzilla.mozilla.org/attachment.cgi?id=153557&action=edit ...

superreview requested: [Bug 321279] Drag+drop of link to location bar broken, anchor node text gets included : [Attachment 239750] don't step on CF_UNICODETEXT with url flavours
Ted Mielczarek (luser) <ted.mielczarek@gmail.com> has asked Robert O'Callahan (Novell) <roc@ocallahan.org> for superreview: Bug 321279: Drag+drop of link to location bar broken, anchor node text gets included https://bugzilla.mozilla.org/show_bug.cgi?id=321279 Attachment 239750: don't step on CF_UNICODETEXT with url flavours https://bugzilla.mozilla.org/attachment.cgi?id=239750&action=edit ...

superreview granted: [Bug 162791] PATCH: URL bar accepts Drag & Drop of strings/URLs : [Attachment 122076] doesn't load URL on drop
Mike Pinkerton <pinkerton@aol.net> has granted Simon Woodside <sbwoodside@yahoo.com>'s request for superreview: Bug 162791: PATCH: URL bar accepts Drag & Drop of strings/URLs http://bugzilla.mozilla.org/show_bug.cgi?id=162791 Attachment 122076: doesn't load URL on drop http://bugzilla.mozilla.org/attachment.cgi?id=122076&action=edit ------- Additional Comments from Mike Pinkerton <pinkerton@aol.net> sr=pink ...

superreview requested: [Bug 40838] disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) : [Attachment 19
Florian QUEZE <f.qu@laposte.net> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 40838: disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) https://bugzilla.mozilla.org/show_bug.cgi?id=40838 Attachment 196055: patch v2 https://bugzilla.mozilla.org/attachment.cgi?id=196055&action=edit ...

superreview requested: [Bug 40838] disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) : [Attachment 20 #2
Florian Qu�ze <f.qu@queze.net> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 40838: disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) https://bugzilla.mozilla.org/show_bug.cgi?id=40838 Attachment 201286: patch v3 https://bugzilla.mozilla.org/attachment.cgi?id=201286&action=edit ------- Additional Comments from Florian Qu�ze <f.qu@queze.net> New patch following the idea "set the status bar text before the event is ...

superreview requested: [Bug 40838] disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) : [Attachment 20 #3
Florian Qu�ze <f.qu@queze.net> has asked Peter Van der Beken <peterv@propagandism.org> for superreview: Bug 40838: disallowing "change status bar text" often leaves status bar empty for links (onmouseover="return true;" --> status bar doesn't show url for links) https://bugzilla.mozilla.org/show_bug.cgi?id=40838 Attachment 201286: patch v3 https://bugzilla.mozilla.org/attachment.cgi?id=201286&action=edit ...

superreview requested: [Bug 261717] Can't copy Website header (URL) in RSS message's header : [Attachment 165556] patch that enables a context menu to copy the URL (now localizable)
David Fraser <davidf@sjsoft.com> has asked Scott MacGregor <mscott@mozilla.org> for superreview: Bug 261717: Can't copy Website header (URL) in RSS message's header https://bugzilla.mozilla.org/show_bug.cgi?id=261717 Attachment 165556: patch that enables a context menu to copy the URL (now localizable) https://bugzilla.mozilla.org/attachment.cgi?id=165556&action=edit ------- Additional Comments from David Fraser <davidf@sjsoft.com> Requesting review for this. It's a fairly straight forward patch, the main thing is whether it's correct to add ...

superreview requested: [Bug 340273] Can't drag proxy icons from tabs to location bar if focus is outside location bar : [Attachment 239703] This patch is the same as above, with an addition to fix a c
Stuart Morgan <stuart.morgan@alumni.case.edu> has asked Simon Fraser <sfraser_bugs@smfr.org> for superreview: Bug 340273: Can't drag proxy icons from tabs to location bar if focus is outside location bar https://bugzilla.mozilla.org/show_bug.cgi?id=340273 Attachment 239703: This patch is the same as above, with an addition to fix a compiler warning not related to this bug https://bugzilla.mozilla.org/attachment.cgi?id=239703&action=edit ------- Additional Comments from Stuart Morgan <stuart.morgan@alumni.case.edu> Looks good, r=me. Thanks! As an added b...

superreview requested: [Bug 313217] document.open('replace') now opens text/plain document where it used to open text/html : [Attachment 200309] Patch to make a type of "replace" mean HTML but not re
Boris Zbarsky <bzbarsky@mit.edu> has asked Johnny Stenback <jst@mozilla.org> for superreview: Bug 313217: document.open('replace') now opens text/plain document where it used to open text/html https://bugzilla.mozilla.org/show_bug.cgi?id=313217 Attachment 200309: Patch to make a type of "replace" mean HTML but not replace https://bugzilla.mozilla.org/attachment.cgi?id=200309&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> Just treat "replace" in the first slot as if nothing were set. ...

superreview requested: [Bug 340273] Can't drag proxy icons from tabs to location bar if focus is outside location bar : [Attachment 239867] Same patch, just moved around the #import.
Samuel Sidler (:sps) <samuel.sidler@gmail.com> has asked Simon Fraser <sfraser_bugs@smfr.org> for superreview: Bug 340273: Can't drag proxy icons from tabs to location bar if focus is outside location bar https://bugzilla.mozilla.org/show_bug.cgi?id=340273 Attachment 239867: Same patch, just moved around the #import. https://bugzilla.mozilla.org/attachment.cgi?id=239867&action=edit ------- Additional Comments from Samuel Sidler (:sps) <samuel.sidler@gmail.com> Carrying over review... (Typically if there's just a small change to make, you can carry o...

superreview requested: [Bug 258175] Drag and drop of url in url bar to bookmarks toolbar doesn't work : [Attachment 159455] fix?
Robert O'Callahan <roc@ocallahan.org> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 258175: Drag and drop of url in url bar to bookmarks toolbar doesn't work http://bugzilla.mozilla.org/show_bug.cgi?id=258175 Attachment 159455: fix? http://bugzilla.mozilla.org/attachment.cgi?id=159455&action=edit ...

Web resources about - superreview requested: [Bug 313027] Can't drag plain text url to tab bar : [Attachment 205462] Patch to make url parsing from text less strict - mozilla.dev.super-review

Biz & Finance Magazines
Australian magazine subscriptions price comparison.

Ehsan Akhgari
Bugzilla is an essential tool to the working process of Mozilla, and many of us spend a good portion of their day in Bugzilla. The Bugzilla Tweaks ...

Resources last updated: 1/10/2016 8:52:20 PM