Bug 120822 - Web Inspector: Breakpoint Action JS Editor has line wrapping issues with long no-space lines
Summary: Web Inspector: Breakpoint Action JS Editor has line wrapping issues with long...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2013-09-05 19:15 PDT by Joseph Pecoraro
Modified: 2013-09-05 20:32 PDT (History)
5 users (show)

See Also:

Attachments
[PATCH] Proposed Fix (1.52 KB, patch)
2013-09-05 19:17 PDT, Joseph Pecoraro 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2013-09-05 19:15:53 PDT 
Go back to fixed width value. Trying to use a % is causing issues with the container expanding on long no-space lines.

Also note, CodeMirror issue: <https://github.com/marijnh/CodeMirror/issues/1809>
Comment 1 Radar WebKit Bug Importer 2013-09-05 19:16:40 PDT 
<rdar://problem/14925829>
Comment 2 Joseph Pecoraro 2013-09-05 19:17:14 PDT 
Created attachment 210693 [details]
[PATCH] Proposed Fix
Comment 3 Joseph Pecoraro 2013-09-05 19:18:16 PDT 
* STEPS TO REPRODUCE
1. Edit Breakpoint
2. Create "Evaluate JavaScript" breakpoint action
3. Type: document.getElementById("something").addEventListener
  => with width:100% the line doesn't wrap, instead the editor grows and messes up the popover
Comment 4 WebKit Commit Bot 2013-09-05 20:32:57 PDT 
Comment on attachment 210693 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 210693

Committed r155169: <http://trac.webkit.org/changeset/155169>
Comment 5 WebKit Commit Bot 2013-09-05 20:32:59 PDT 
All reviewed patches have been landed.  Closing bug.