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

feat: about section on subdomain loads #263

Merged
merged 2 commits into from
May 21, 2024
Merged

Conversation

lidel
Copy link
Member

@lidel lidel commented May 20, 2024

This PR moves "about" into separate component and reuses it on subdomain landing page displayed when SW is not registered yet.

This aims to fix UX problem when I send https://en-wikipedia--on--ipfs-org.ipns.inbrowser.dev to someone, and all they see is no wikipedia, but a confusing button to click, and some configuration, but have no idea what is happening.

Feedback welcome, preview below:

Before

2024-05-20_16-38

After

2024-05-20_16-28

@lidel lidel requested review from SgtPooki and 2color May 20, 2024 14:41
@lidel lidel added this to the Beta milestone May 20, 2024
@lidel lidel force-pushed the feat/about-on-subdomains branch from 05ca132 to 55879c1 Compare May 20, 2024 14:59
Without this people see  confusing button to click, and some
configuration, but have no idea what is happening.
@lidel lidel force-pushed the feat/about-on-subdomains branch from 55879c1 to d86c1ae Compare May 20, 2024 15:06
Copy link
Member

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

awesome

@lidel lidel merged commit 947011e into main May 21, 2024
16 checks passed
@lidel lidel deleted the feat/about-on-subdomains branch May 21, 2024 00:56
@SgtPooki SgtPooki mentioned this pull request Jun 12, 2024
2color added a commit that referenced this pull request Jun 14, 2024
* chore(main): release 1.2.0 (#216)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat: about section on subdomain loads (#263)

Without this people see  confusing button to click, and some
configuration, but have no idea what is happening.

Co-authored-by: Russell Dempsey <1173416+SgtPooki@users.noreply.github.com>

* fix: remove fonts except woff2 (#282)

* fix: add purgecss webpack plugin

* fix: keep only woff2 font, remove purgeCss

* chore: improve LCP performance (#293)

---------

Co-authored-by: Marcin Rataj <lidel@lidel.org>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Daniel Norman <1992255+2color@users.noreply.github.com>
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