Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added limits for size, an error message popup and a cursor #185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aaravtanti
Copy link

Changes

  • Added a cursor to the size box
  • Added a limit for the range on 1-1000 for the size box
  • Added an error message if user input exceeds 1000
  • Added functionality that does not allow the user to have less than 1 digit in the text box

Why (resolves #184)

  • Addressed the concern of the difficulty in typing a new size for the bars by implementing cursor functionality
  • Added functionality that does not allow the user to have less than 1 digit so that the program does not crash (as there will never be no input in the size box)

How

  • Added an error message using the pygame display function; set a timer for the error message to ensure that the viewer can quickly change their input
  • Added cursor definitions in the class and updated the draw function

Testing
nil

Screenshots

Clipped.for.PR.mov

@aaravtanti aaravtanti changed the title Added limits for size and an error message popup Added limits for size, an error message popup and a cursor Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: sizeBox should not allow null input
1 participant