-
-
Notifications
You must be signed in to change notification settings - Fork 875
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
Make OSD screen bigger if possible #2916
Make OSD screen bigger if possible #2916
Conversation
This comment has been minimized.
This comment has been minimized.
@McGiverGim this is my result lol: |
What about a possibility to change the OSD background from the IDE ? |
@asizon What screen resolution are you using? I think I forgot maybe give a min-size to the display. I will test it later.
@HThuren we have a petition of not do it :P betaflight-configurator/src/css/tabs/osd.css Lines 464 to 466 in 6319421
But I think you can open a feature request for this. This petition is out of the scope of this PR. |
@McGiverGim not sure about my screen resolution, but is small, i can check it later. |
4e5f4a3
to
68612ff
Compare
@asizon modified the code. Changed the "manual" adjustment in JS to a lot of FLEX magic in CSS. Now I hope it adjusts to your resolution... |
This comment has been minimized.
This comment has been minimized.
79e473c
to
f684092
Compare
Modified, rebased and cleaned. Ready for a real review and test. |
This comment has been minimized.
This comment has been minimized.
AUTOMERGE: (FAIL)
|
f684092
to
10a49a4
Compare
@benlumley thanks for the feedback... Pushed a new version, I've adjusted the size of the columns to fit in the space just before the collapse of the menu. Is a little ugly, the texts and the selects of the OSD lateral columns wrap, but all remains in correct position. There is not too much that we can do with this small resolution. EDIT: another solution is maybe collapse the menu a little earlier... |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't test OSX otherwise it's LGTM
Need to refresh my skills: https://developer.mozilla.org/en-US/docs/Web/CSS/flex-basis
Found a little hickup for this solution. Min-Width of the window is limited as well as min-height of window but min-height limiting is to small, because OSD image is cropped. Min-height limitation should be a little bit bigger. Will include a screenshoot asap |
@mrRobot62 what resolution is this? I will change it to take into account the height of the window. |
@McGiverGim - take a look at the left border, the menu is cropped too. The context area should be a little bit smaler, to see the menu icons (without text) but icons should visible completely. That's more user friendly |
A little magic in the CSS and now it adjusts to the height too... at the minimum height it is a little out of screen, but it was before this change too... the only solution is to limit a little more the minimum resolution. |
This comment has been minimized.
This comment has been minimized.
3e32eaf
to
15ea460
Compare
Pushed new version with the little fixes of latest review. |
This comment has been minimized.
This comment has been minimized.
@McGiverGim please rebase |
15ea460
to
aa8ed7d
Compare
aa8ed7d
to
ec7822f
Compare
Kudos, SonarCloud Quality Gate passed!
|
Do you want to test this code? Here you have an automated build: |
Works well on my Mac! Looks great - thank you! |
Use the scroll to center the OSD. Of course limiting the size is another option but it's not necessary at this moment. |
Fixes #2631
This is an attempt to make the OSD scale with large resolutions. This is the first versions, it needs:
I publish it in this form to get some feedback and testing.
This is an example of before:
![image](https://user-images.githubusercontent.com/2673520/167287196-6a91cf53-4a98-4f06-9497-d1643afa2711.png)
And after:
![image](https://user-images.githubusercontent.com/2673520/167287111-78ee7785-6457-4477-9d0d-117c48e9f04d.png)