Skip to content

Calculator: The C (Clear) button in the unit converter only deletes one character instead of clearing the input #338

Description

@fubai578

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.4.0 (version code 10)

Affected Android/Custom ROM version

Android 14 (API 34)

Affected device model

Android emulator, Pixel 6 profile (1080 x 2400)

How did you install the app?

GitHub releases

Steps to reproduce the bug

  1. Open the app and go to Unit converter.
  2. Select Speed and keep the default Kilometer per hour -> Mile per hour.
  3. Type 6 then 0 on the number pad. The input shows 60 and the result shows 37.282271534240038.
  4. Tap the C button (resource id org.fossify.math:id/btn_clear).
  5. Look at the input field and the result field again, both immediately and after the page settles.

Expected behavior

After tapping C, both the input and the converted result should return to the neutral zero state (0 / 0). If the intended behaviour really is to delete a single character, the button should be a backspace (⌫) rather than being labelled C (Clear) - the label and the behaviour should match, or the wording should be updated so it does not conflict with what users expect from a Clear key.

Actual behavior

With 60 in the input, the result field shows 37.282271534240038. After tapping C, the input only becomes 6 and the result becomes 3.728227153424004 - the key removed the last digit instead of clearing the input. Keeping 60 without tapping C on the same Speed route shows the input and result refreshing normally, so the problem is limited to the clear action.

Screenshots/Screen recordings

ignoreImageMinify ignoreImageMinify
01_trigger_clear.mp4

Additional information

Package: org.fossify.math
Version: 1.4.0.

The FAQ says the C button removes one character at a time and that long pressing resets all fields, so the button already does what it is meant to. My problem is the label: C reads as Clear, so I expected it to empty the input and was surprised when it only removed the last digit.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething is not workingneeds triageIssue is not yet ready for PR authors to take up

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions