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.

Sign up for GitHub

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

Jump to bottom

Refresh of index page #327

Merged
merged 7 commits into from
Aug 27, 2023
Merged

Refresh of index page #327

merged 7 commits into from
Aug 27, 2023

Conversation

Hans5958
Copy link
Member

@Hans5958 Hans5958 commented May 4, 2023 •
edited
Loading

EDIT: This preview is outdated, please check the latest image.

Was going to update the description/mission statement, but then had an idea on do a revamp of it. Let me know what you think.

Untitled

@Hans5958 Hans5958 added type: enhancement New feature or request scope: content Related to the contents of the website, excluding blog and docs labels May 4, 2023
@Secret-chest
Copy link
Contributor

Too much blank space to me. We should probably first remove the curve, then add another layer with the download buttons.

@Hans5958
Copy link
Member Author

Hans5958 commented May 5, 2023 •
edited
Loading

A little bit changes, now with a static hero image, with the slide images on the bottom. I have to consider the circumstances when it needs updating. Probably a better one would be the settings page, but what do you guys think?

EDIT: The hero image is sourced from the new webstore images concept few months ago. ScratchAddons/ScratchAddons#4811 (comment)

image

@WorldLanguages
Copy link
Member

I like the "hero image". It's easy on the eyes and users immediately understand that we're (mainly) a browser extension. And I like that my eye only has to focus on a specific part of the image.
I would consider highlighting more than 1 enabled addon. Possibly 2 enabled and the last one disabled (to be considered - the popup does not show disabled addons and that could confuse users).

@WorldLanguages
Copy link
Member

Would also be great if clicking "load more" did smooth-scrolling instead of abruptly sending you to the "features" section.

@Secret-chest
Copy link
Contributor

I think we should show more images, but not scroll them automatically.

@Hans5958
Copy link
Member Author

we should show more images, but not scroll them automatically.

User may miss the images if it isn't automatically scroll. Don't worry, it would stop if someone interacts with it (which is still the current behavior live on the site)

@Hans5958 Hans5958 marked this pull request as draft August 4, 2023 11:27
@Hans5958
Copy link
Member Author

Hans5958 commented Aug 17, 2023 •
edited
Loading

TODO (In case anyone want to pick this up)

  • Do a better mockup for the hero image. For example, we can update the dummy browser to be something more... newer. Original .svg attatched:
    intro hero image

@Samq64
Copy link
Member

Samq64 commented Aug 17, 2023 •
edited
Loading

Do a better mockup. We can update the dummy browser to be something more... newer.

Maybe the popup could also show a few more addons?

@Hans5958
Copy link
Member Author

Hans5958 commented Aug 17, 2023 •
edited
Loading

Maybe the popup could also show a few more addons?

Oh, yes, of couse, the one improvement I mentioned is just an example.

@Hans5958 Hans5958 added the help wanted Extra attention is needed label Aug 26, 2023
@Hans5958
Update intro/mission statement
0b7ec2c
@Hans5958
Change style of landing intro
da3f973 
Responsife font size (inspired with Bootstrap)
Make font size larger
Have centered one column design
@Hans5958
Use static hero image, put old images on features section
9cebf22
@Hans5958 Hans5958 force-pushed the landing-intro-refresh branch from 3792b2a to 9f19da4 Compare August 26, 2023 05:24
@Hans5958 Hans5958 force-pushed the landing-intro-refresh branch from 9f19da4 to 23b8db1 Compare August 26, 2023 05:26
@Hans5958
Copy link
Member Author

Hans5958 commented Aug 26, 2023 •
edited
Loading

Got time around to update the hero image. While the browser is still... that, I tried to tidy up it, while updating the images and add the Scratch website (blurred in case of problems like copyright, and to divert focus to the extension popup). This should make it ready to merge.

2023-08-26_12-31-24

@Hans5958 Hans5958 marked this pull request as ready for review August 26, 2023 05:33
@Secret-chest
Copy link
Contributor

I don't like the blur, it is disrupting and the Scratch screenshot is vanilla anyways.

@Hans5958 Hans5958 force-pushed the landing-intro-refresh branch from 8cfcebe to 28f828e Compare August 26, 2023 06:27
@Hans5958 Hans5958 mentioned this pull request Aug 26, 2023
Draft
@WorldLanguages
Copy link
Member

WorldLanguages commented Aug 26, 2023 •
edited
Loading

Looks very nice!

If I could change one more thing about the landing page, it would probably be the Satisfied? Don't hesitate to install it now! phrase. It doesn't really fit our website, I think.

@Hans5958
Copy link
Member Author

Hans5958 commented Aug 27, 2023 •
edited
Loading

What do you mean by "doesn't really it"?

Any ideas? I think I would remove the "Satisfied?" part first, but I would leave it as-is for now.

@Hans5958 Hans5958 changed the title Refresh of landing/index intro Refresh of index page Aug 27, 2023
@Hans5958 Hans5958 merged commit d4524a0 into master Aug 27, 2023
@Hans5958 Hans5958 mentioned this pull request Aug 27, 2023
Closed
@BroJac5246
Copy link
Contributor

Looks very nice!

If I could change one more thing about the landing page, it would probably be the Satisfied? Don't hesitate to install it now! phrase. It doesn't really fit our website, I think.

Just want to add that this coincidentally occurred to me today as well; it looks out-of-place and is somewhat unprofessional.

@Hans5958 Hans5958 deleted the landing-intro-refresh branch August 27, 2023 05:37
@Hans5958 Hans5958 mentioned this pull request Aug 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WorldLanguages WorldLanguages WorldLanguages approved these changes

Assignees
No one assigned
Labels
help wanted Extra attention is needed scope: content Related to the contents of the website, excluding blog and docs type: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Hans5958 @Secret-chest @WorldLanguages @Samq64 @BroJac5246