Opened 11 years ago

Last modified 4 years ago

#9944 new enhancement

DeskCalc : separating keypad labels from button codes

Reported by: SoulHunter Owned by: stippi
Priority: low Milestone: R1.1
Component: Applications/DeskCalc Version: R1/alpha4.1
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Here is a little enhancement to the DeskCalc app :

It originally couldnt use alternative labels (math symbols -except pi letter-) for the keypad because every label is used "as is" and passed directly to the math parser.

When I discussed with jscipione, and after some code browsing we suggested using another a new field to store the key "function". The thing is there is already a field called "code" and used to store the label except for the key "=" which is replaced with "\n", so I assumed that the person who made that wanted some separation but did it for one key. This naturally got me into using the "code" field and treat the return key like any another one.

The details about how it works : in the keypad description string, a separator char (I chose the '|') is used between the "code" and the "actual label", if there is no separator given, the same code is assumed to be the label.

Regards.

Attachments (1)

deskcalc.patch (9.2 KB ) - added by SoulHunter 11 years ago.
diff patch

Download all attachments as: .zip

Change History (4)

by SoulHunter, 11 years ago

Attachment: deskcalc.patch added

diff patch

comment:1 by SoulHunter, 11 years ago

patch: 01

comment:2 by vidrep, 7 years ago

Has anybody ever reviewed this patch?

comment:3 by pulkomandy, 4 years ago

Milestone: R1R1.1
Note: See TracTickets for help on using tickets.