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.

Sign up for GitHub

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

Jump to bottom

Impossible to read error message #9808

Closed
ceraolo opened this issue Nov 29, 2022 · 0 comments · Fixed by #11133
Closed

Impossible to read error message #9808

ceraolo opened this issue Nov 29, 2022 · 0 comments · Fixed by #11133
Assignees
@adeas31
Labels
COMP/GUI/OMEdit Issue and pull request related to OMEdit
Milestone
1.22.0

Comments

@ceraolo
Copy link

ceraolo commented Nov 29, 2022 •
edited

Description

The message browser has not a horizontal scrollbar, and some error messages do not wrap text. The result is that we cannot read these error messages in full

Steps to Reproduce

  1. load Test.mo: Test.mo.txt
  2. run Id3Pwm1

You'll get the following output on the message browser:
image

As you see, large part of the red error (indeed it should be warning, since the simulation recovers and finishes correctly, and therefore shouldn't be red) is impossible to read, because goes beyond the right border of the window.

Expected Behavior

  1. the error message should be wrapped
  2. messages indicating issues that are successfully overcome, like this ticket's, should be better written in the color used for notifications, at least for the part that reports a solver switch to the fallback one. Maybe "matrix singular" could stay red (not sure about this).

Screenshots

Version and OS

OM 1.20-beta4 Win10

Additional Context
@ceraolo ceraolo mentioned this issue Nov 30, 2022
Closed
@casella casella added the COMP/GUI/OMEdit Issue and pull request related to OMEdit label Nov 30, 2022
@casella casella added this to the 1.21.0 milestone Nov 30, 2022
@casella casella modified the milestones: 1.21.0, 1.22.0 Apr 18, 2023
@adeas31 adeas31 modified the milestones: 1.22.0, UniPI DFD - September 2023 Jun 16, 2023
@adeas31 adeas31 mentioned this issue Sep 7, 2023
Closed
adeas31 added a commit to adeas31/OpenModelica that referenced this issue Sep 7, 2023
@adeas31
Use default wrap mode i.e., QTextOption::WrapAtWordBoundaryOrAnywhere
056b874 
Do not use QTextOption::WordWrap
With QTextOption::WrapAtWordBoundaryOrAnywhere, if possible, wrapping occurs at a word boundary; otherwise it will occur at the appropriate point on the line, even in the middle of a word.

Fixes OpenModelica#9808 and fixes OpenModelica#11118
@adeas31 adeas31 mentioned this issue Sep 7, 2023
Merged
adeas31 added a commit that referenced this issue Sep 7, 2023
@adeas31
Use default wrap mode i.e., QTextOption::WrapAtWordBoundaryOrAnywhere (…
682fa8b 
…#11133)

Do not use QTextOption::WordWrap
With QTextOption::WrapAtWordBoundaryOrAnywhere, if possible, wrapping occurs at a word boundary; otherwise it will occur at the appropriate point on the line, even in the middle of a word.

Fixes #9808 and fixes #11118
@casella casella modified the milestones: UniPI DFD - October 2023, 1.23.0, 1.22.0 Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adeas31 adeas31

Labels
COMP/GUI/OMEdit Issue and pull request related to OMEdit
Projects
2023 UniPI DFD
Status: Done
Milestone
1.22.0
Development

Successfully merging a pull request may close this issue.

Use default wrap mode i.e., QTextOption::WrapAtWordBoundaryOrAnywhere adeas31/OpenModelica
3 participants
@adeas31 @casella @ceraolo