[Bug 637189] New: webui/live_build_log: uses weird space characters
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c0 Summary: webui/live_build_log: uses weird space characters Classification: openSUSE Product: openSUSE.org Version: unspecified Platform: All OS/Version: Linux Status: NEW Severity: Normal Priority: P5 - None Component: BuildService AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: jengelh@medozas.de QAContact: adrian@novell.com Found By: Beta-Customer Blocker: --- The live build log program (/package/live_build_log) substitutes spaces by U+2002 ('EN SPACE'). That is wrong for several reasons: - the space may not coincide with the width of an ASCII space (U+0020) - U+2002 is not a stop character, so trying to mark words with the mouse will mark multiple words - copy-and-paste is useless in the sense that the paste target will have U+2002 instead of the desired ASCII space, leading to all sorts of strange errors. (Shell splitting at whitespace is one thing that is inhibited) - etc. The output of "osc buildlog" is not affected, so I see this as a webui problem. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c1 --- Comment #1 from Jan Engelhardt <jengelh@medozas.de> 2010-09-04 17:24:52 UTC --- Only concerns <div id="log_space_wrapper">, the remainder of the HTML page is unaffected. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c n liu <nliu@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nliu@novell.com AssignedTo|bnc-team-screening@forge.pr |adrian@novell.com |ovo.novell.com | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c Adrian Schröter <adrian@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|BuildService |webui Product|openSUSE.org |openSUSE Build Service -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c2 Jan Engelhardt <jengelh@medozas.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |coolo@novell.com --- Comment #2 from Jan Engelhardt <jengelh@medozas.de> 2011-02-28 09:19:35 UTC --- 10:04 < coolo> sqft: U+2002 is 'EN SPACE' 10:06 < sqft> coolo: why can't it use U+A0? 10:06 < coolo> sqft: well, this does not break 10:07 < coolo> sqft: U+20 is eaten by HTML parsers 10:13 < sqft> s{^ }{\x{a0}};s{ }{ \x{a0}}g Not eaten, does break, and can be copied from browser to shell window without having any non-U+20 in the terminal afterwards. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c Adrian Schröter <adrian@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|adrian@suse.com |speilicke@suse.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c3 Sascha Peilicke <speilicke@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |vljn@ovi.com --- Comment #3 from Sascha Peilicke <speilicke@suse.com> 2012-04-02 09:46:34 UTC --- *** Bug 629404 has been marked as a duplicate of this bug. *** http://bugzilla.novell.com/show_bug.cgi?id=629404 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c4 Sascha Peilicke <speilicke@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED --- Comment #4 from Sascha Peilicke <speilicke@suse.com> 2012-04-02 11:02:44 UTC --- The respective code currently contains: text.gsub!(/^ /, " ") text.gsub!(/ /, " ") which is the same as sqft recommended. One might argue if this computation is actually worth it or whether we should replace the <code> tag in the template with a <pre> plus some CSS including overflow and white-space rules: pre { overflow-x: auto; /* Use horizontal scroller if needed; for Firefox 2, not needed in Firefox 3 */ white-space: pre-wrap; /* css-3 */ white-space: -moz-pre-wrap !important; /* Mozilla, since 1999 */ white-space: -pre-wrap; /* Opera 4-6 */ white-space: -o-pre-wrap; /* Opera 7 */ /* width: 99%; */ word-wrap: break-word; /* Internet Explorer 5.5+ */ } Closing as fixed in the meantime... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c5 Sascha Peilicke <speilicke@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #5 from Sascha Peilicke <speilicke@suse.com> 2012-04-02 11:50:07 UTC --- Fixed with git commit 0657cdc -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c6 Sascha Peilicke <speilicke@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |szotsaki@gmail.com --- Comment #6 from Sascha Peilicke <speilicke@suse.com> 2012-04-02 13:41:46 UTC --- *** Bug 637000 has been marked as a duplicate of this bug. *** http://bugzilla.novell.com/show_bug.cgi?id=637000 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=637189 https://bugzilla.novell.com/show_bug.cgi?id=637189#c7 Jan Engelhardt <jengelh@medozas.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |VERIFIED --- Comment #7 from Jan Engelhardt <jengelh@medozas.de> 2012-04-03 12:36:51 UTC --- The fix is actually commit 5511a17754fe0b26ec6df1ef1b1cc00dba7ac20a. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com