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

Fix prompt labels overflows in create order #5251

Closed
wants to merge 2 commits into from
Closed

Fix prompt labels overflows in create order #5251

wants to merge 2 commits into from

Conversation

forcryptos
Copy link

Fixes #5189

This change is to fix the prompt labels that where overflowing
outside the InputTextBoxes in the CreateOffer section (Buy and Sell).

  • Created a class StringUtils providing truncatePromptString.
  • Call truncatePromptString in MutableOfferView and in MutableOfferViewModel.

I've created a class StringUtils located in common/util.

StringUtils.truncatePromptString is used five times in
MutableOfferView.java directly on the prompt value.

It is also used once in MutableOfferViewModel.java for the
volume prompt which is a interpolated string
@boring-cyborg
Copy link

boring-cyborg bot commented Feb 28, 2021

Thanks for opening this pull request!

Please check out our contributor checklist and check if Travis or Codacy found any issues with your PR. Also make sure your commits are signed, and that you applied Bisq's code style and formatting.

A maintainer will add an is:priority label to your PR if it is up for compensation. Please see our Bisq Q1 2020 Update post for more details.

@forcryptos
Copy link
Author

I am honestly not sure if there was a simpler solution to fix this. Maybe someone will know more about javafx and its CSS rules and can suggest a better fix.

@forcryptos forcryptos closed this Feb 28, 2021
@forcryptos forcryptos reopened this Feb 28, 2021
@forcryptos forcryptos closed this Feb 28, 2021
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Trigger price truncation of prompt text not working
1 participant