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

[Install pages] Adapt support table width to full container width #253

Merged
merged 12 commits into from
Mar 5, 2023

Conversation

eLtMosen
Copy link
Member

@eLtMosen eLtMosen commented Mar 2, 2023

The support table was fixed width and looked off after adding the model number table.
This PR

  • Makes the support table behave visually like the model number table
  • Is tailored to our current max 15 items worst case table for beluga and catfish

This method prevents horizontal scrolling down to a minimum screen width of 520px.

Before

support-table-before-2023-03-02_14.31.02.mp4

After, for current worst case catfish

support-table-after-2023-03-02_14.59.07.mp4

@eLtMosen eLtMosen marked this pull request as draft March 2, 2023 14:54
@FlorentRevest
Copy link
Member

Thanks for all the efforts and recording the videos and everything :D

@eLtMosen
Copy link
Member Author

eLtMosen commented Mar 3, 2023

Downsides needing fix:

  • Due to tailoring to catfish/beluga, watches with fewer features have their titles shrunk unnecessarily small
  • This sort of hardcoded and would need adaption as soon as a new feature is added.
  • Even on "optimized" catfish/beluga, the support tableis only responsive down to 520px width. Looking awfull on phones still

@eLtMosen
Copy link
Member Author

eLtMosen commented Mar 3, 2023

New design to remove the downsides of previuos approach.

  • Feature name moved into to feature status div
  • Colors adapted to work as background color for black text
  • Feature boxes keep the size and break into rows when the screen width is reduced

grafik

grafik

@eLtMosen eLtMosen marked this pull request as ready for review March 3, 2023 19:54
@FlorentRevest
Copy link
Member

Looks very cool, great work! Thank you @eLtMosen

@FlorentRevest FlorentRevest merged commit d6159f8 into AsteroidOS:master Mar 5, 2023
@eLtMosen
Copy link
Member Author

eLtMosen commented Mar 5, 2023

Looks very cool, great work! Thank you @eLtMosen

@dodoradio greatly helped to polish it!

@FlorentRevest
Copy link
Member

FlorentRevest commented Mar 5, 2023

Thanks dodo too!

Just a note, it seems that you introduced a stray "Preparation" title before "Choose your variant" in the MTK6580 watches page https://asteroidos.org/install/harmony/

@FlorentRevest
Copy link
Member

https://asteroidos.org/install/beluga/ also has a stray "Hardware Support" :)

@eLtMosen eLtMosen deleted the support-table-full-width branch March 7, 2023 11:03
eLtMosen added a commit to eLtMosen/asteroidos.org that referenced this pull request Mar 9, 2023
…teroidOS#253)

* Introduce table-support class with responsive behaviour tailored to the current max 15 item support table

* Assign new table-support class to all install pages

* [aw-install-dual] Implement new div based design for the support table

* [aw-install-multi] Implement new div based design for the support table

* [aw-install] Implement new div based design for the support table

* [aw-install-simg] Implement new div based design for the support table

* [beluga-install] Implement new div based design for the support table

* [mtk-install] Implement new div based design for the support table

* [mooneye-install] Implement new div based design for the support table

* Add install-support-col, define new feature status div, and adapt status colors to work as text background

* Remove border and fix tool tips visibility vs display to not occupy space when hidden
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.

2 participants