-
-
Notifications
You must be signed in to change notification settings - Fork 737
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
WIP: Split Render for multiple screen resolutions #1888
Merged
+2,052
−1,588
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
f58d7df
Create README.md
Ralim 6a0a19f
Move to new folder
Ralim 9ff4805
Migrating
Ralim d4c1805
Migrate Remainder
Ralim dcfb0b9
format fix (all but one) (#1889)
discip df81e88
Update USBPDDebug_FS2711.cpp
Ralim 6aa4e6a
Delete PrintVoltage.cpp
Ralim d8d9663
Copy in 128x32 template
Ralim 976d90c
Mask drawing for 96x16
Ralim 5a69829
Import #1819
Ralim 3a111d2
Update Font.h
Ralim ca2485c
Homescreen
Ralim fe534c9
Update draw_homescreen_detailed.cpp
Ralim 5f48b53
Fix oled normal draw for variable height
Ralim bf36fd6
Update OLED.cpp
Ralim 95a85e6
Draw settings icons
Ralim dea759f
Update draw_homescreen_simplified.cpp
Ralim ccdbc80
Update draw_power_source_icon.cpp
Ralim 2450b65
Fixup oled drawing for fill area
Ralim 8e48988
Update the region fill for mixed heights
Ralim cef0ee3
Fix newline height
Ralim f436574
FIXUP! Draw icons in settings menu at correct size
Ralim ab930c4
Fix scrollbar
Ralim 7b18975
Update settingsGUI.cpp
Ralim a9ecbca
S60(P) Disable auto display rotation
Ralim c45c77e
On tall oled, scroll in 2 line increments
Ralim 9e51a9c
Bugfix transition L<->R
Ralim 2cb27c9
Draw every other one on transitions
Ralim fbe4633
.
Ralim c53ab37
cleanup
Ralim 6159aa7
Bootup logo: Draw in centre
Ralim e5a917b
Merge branch 'dev' into gui-multi-screen
discip e16a2d4
Update OLED.hpp
discip File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Just from a parametric perspective:
Could we use something like this here (instead of static values):
https://github.com/Ralim/IronOS-dfu/blob/7ab0a57c49e1913b87b49ae458e641aa48adea87/src/oled.c#L168
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.
At the moment the whole bootlogo stuff is setup only for that sizing. Which is sort of why I left it like that. 🤔
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.
Just curious? 😊
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.
Very valid question.
Still on the fence for what to do about the boot logo stuff.
Since limited frames could be messy, but also nice to do static full screen at the least.
But having half-half also feels messy.
So maybe I just carve out a little more logo space on these devices to have at least some frames of boot logo?
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.
Inclined to leave it as-is for the super short term as there are probably bigger issues to solve 🙃