Frontend responsive fontsize android #3623
Draft
Conversation
This file contains hidden or 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
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
from
46039ca to
e8ddfec
Compare
Collaborator
Author
|
Will replace #2699 |
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
4 times, most recently
from
c2d1d9e to
d5518bc
Compare
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
5 times, most recently
from
fab6678 to
044e95e
Compare
Merged
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
2 times, most recently
from
4ca695a to
2f600cb
Compare
Merged
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
9 times, most recently
from
e2f6c6d to
d47a4e5
Compare
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
5 times, most recently
from
f6df564 to
bee0d54
Compare
As there are many components that have a font-size between small and default, we need to rename to smaller first. In a second step size small can be reintroduced and used for all the components mentioned before. This can be done in the next commit.
Use CSS variable --size-small for formerly hardcoded font-size that was between --size-default and --size-small (now --size-smaller).
Use font-sizes with defined maximum values so that the layout does not break if 'Font size' is increased in Android setting.
Bottom-menu should not grow beyond certain size in responsive mode for example if Android Font size is set to max. This can be simulated by changing the font-size on the root HTML element to 150%.
Toggles in responsive mode looked broken if Android Font size is increased. This can be simulated by changing the font-size on the root HTML element to 150%.
- changed button border-width to be reponsive and scale with the ui - fixed send/recieve button, should scale - fixed responsive button height
Improved various form elements so they look ok in scaled up mode. Can be tested by setting font-size: 150%; on the html element.
Fixed a bug caused by empy paragraph element, which increased settings item height unnecessarily.
Changed the layout slightly on small screen as there was custom font-size defined, introduced in: bfc3fed
Also removed some unused CSS.
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
from
c6bfd27 to
2c8f42b
Compare
Sidebar can appear on Android tablets, fixed some styles related to the sidebar component. Also removed anmiation all (transition) on sidebar labels. Kept only icons should animate opacity.
Visually changed search input height also in default mode, so it uses the same height as all other input fields.
Updated account-overview styling so it aligns with changes from the last commit (send). Changed header styling, so that the title and balance positioning match between account and send view.
The copy component should grow to improve accessibility. Also removed utility classes in favor of already exsting container class.
thisconnect
force-pushed
the
frontend-responsive-fontsize-android
branch
from
2c8f42b to
d64fe39
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Before asking for reviews, here is a check list of the most common things you might need to consider: