Bug 122386 - Edit label with \t in text causes problem
Summary: Edit label with \t in text causes problem
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-21 00:22 UTC by Andrew Walker
Modified: 2006-02-24 16:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Walker 2006-02-21 00:22:00 UTC
Version:           HEAD (using KDE KDE 3.5.0)
Installed from:    Compiled From Sources
OS:                Linux

PROBLEM:
The text entered into an "Edit Label" dialog is not always preserved faithfully.

STEPS TO REPRODUCE:
Start kst
In the default create a label
Edit the label text to read \theta
Hit OK
Enter layout mode and double click on the label to edit

EXPECTED RESULTS:
The text should read \theta

RESULTS:
The text actually reads <tab>heta
Comment 1 Andrew Walker 2006-02-21 00:46:33 UTC
Obviously a problem for \n too.
Comment 2 George Staikos 2006-02-24 16:09:02 UTC
SVN commit 513171 by staikos:

A bunch of label work mostly derived out of planck requirements but generally
quite useful.

1) Invalidate labels when the reference scalars/vectors/strings/expressions
change.  This makes realtime top-level labels work, assuming the data they're
pulling from is updated.  That part will be fixed separately and this patch
will be reduced as a result.
CCBUG: 106307
2) Don't play around with \n and \t.  Our new editor supports them properly so
we can handle both real CR and Tab, and \n and \t.  Fixes munging of labeltext
BUG: 122386
3) Add preliminary support for \textit{}, \textbf{}, \underline{} to the parser
and renderer, and a stub for \textcolor{}{} to the parser.  In general the first
three work, but cannot be nested.  The last one does not work yet since there is
no color parser.  Testcases to follow once nesting is handled.


 M  +110 -1    libkstapp/kstviewlabel.cpp  
 M  +12 -1     libkstapp/kstviewlabel.h  
 M  +43 -6     libkstapp/labelrenderer.cpp  
 M  +6 -0      libkstapp/labelrenderer.h  
 M  +60 -12    libkstmath/labelparser.cpp  
 M  +3 -0      libkstmath/labelparser.h