Bug 85701

Summary: kst equations need modulo (%) operator
Product: [Applications] kst Reporter: Matthew Truch <matt>
Component: generalAssignee: kst
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version: 1.x   
Target Milestone: ---   
Platform: RedHat Enterprise Linux   
OS: Linux   
Latest Commit: Version Fixed In:

Description Matthew Truch 2004-07-22 17:40:45 UTC
Version:           0.99devel (using KDE KDE 3.2.1)
Installed from:    RedHat RPMs
OS:                Linux

equations should have the modulo operator as a valid operator.
Comment 1 George Staikos 2004-07-22 21:53:26 UTC
CVS commit by staikos: 

Implement modulo
CCMAIL: 85701-done@bugs.kde.org


  M +1 -0      enodefactory.cpp   1.4
  M +1 -0      enodefactory.h   1.4
  M +21 -0     enodes.cpp   1.13
  M +1 -0      enodes.h   1.8
  M +94 -86    eparse.c   1.7
  M +3 -2      eparse.h   1.5
  M +3 -1      eparse.y   1.7
  M +66 -59    escan.c   1.5
  M +2 -0      escan.l   1.5



Comment 2 Matthew Truch 2004-07-23 00:36:54 UTC
Grea work, and what speed.  However, the % has not been added to the operator list (pull down menu in the New Equation dialog), so only people dorky enough to check the cvs logs knew it was implimented.  
Comment 3 George Staikos 2004-07-23 01:32:46 UTC
CVS commit by staikos: 

We need a better solution for this
CCMAIL: 85701-done@bugs.kde.org


  M +5 -0      eqdialog.ui   1.24


--- kdeextragear-2/kst/kst/eqdialog.ui  #1.23:1.24
@@ -220,4 +220,9 @@
                     <item>
                         <property name="text">
+                            <string>%</string>
+                        </property>
+                    </item>
+                    <item>
+                        <property name="text">
                             <string>^</string>
                         </property>