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

Enhance forms and other improvements #362

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Enhance forms and other improvements #362

merged 1 commit into from
Nov 20, 2023

Conversation

cosmicproc
Copy link
Contributor

Current home page

image1_original

Proposed home page

image1_enhanced

Current home page on mobile

Proposed home page on mobile

drawing

Current account creation form

image2_original

Proposed account creation form

When additional information unfolded

image2_enhanced_2

Current login form

Proposed login form

Current device pairing page

image5_original

Proposed device pairing page

image5_enhanced
Other forms on the website will look similar as well.


Additional ideas or opinions are welcome.

if (createFormContent.contains(this.parentElement)) {
createFormContent.hidden = true;
}
else {
Copy link
Member

Choose a reason for hiding this comment

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

Code style nit:

...
} else {
...

is preferred, as also used in other portion of this js file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Missed that one

@quh4gko8
Copy link
Member

The css seems broken for Desktop Mode at Chromium-based browsers in Android. The fit-content doesn't seem to take into account of header's width. See the following picture below for reference:
image

@cosmicproc
Copy link
Contributor Author

cosmicproc commented Nov 16, 2023

This should fix the issue.

@thestinger thestinger merged commit c79345e into GrapheneOS:main Nov 20, 2023
2 checks passed
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.

3 participants