processing / p5.js-web-editor

The p5.js Editor is a website for creating p5.js sketches, with a focus on making coding accessible and inclusive for artists, designers, educators, beginners, and anyone else! You can create, share, or remix p5.js sketches without needing to download or configure anything.
https://editor.p5js.org
GNU Lesser General Public License v2.1
1.37k stars 1.32k forks source link

Megathread: issues with "About" modal responsive design #2701

Open lindapaiste opened 10 months ago

lindapaiste commented 10 months ago

Increasing Access

We want the content to be clear and readable on all screen sizes.

Feature enhancement details

We have a bunch of open issues related to the responsive design of the "About" modal. I'm worried that fixing them one-by-one will just create more problems. We need a more substantial rewrite of this CSS and HTML.

Current issues:

Proposed solutions:

How you can help: Post some screenshots here of how you think the modal should look at various breakpoints, including landscape mode. You can play around with the HTML and CSS in your browser's dev tools or you can edit a screenshot in a graphics program. Let's not make PRs until we have a clearer vision of the desired behavior.

aryanas159 commented 10 months ago

@lindapaiste Can I be assigned to this issue?

aryanas159 commented 10 months ago

image How does this layout look for a mobile breakpoint?

lindapaiste commented 10 months ago

@lindapaiste Can I be assigned to this issue?

At this stage we looking for feedback and ideas from the community. We are not ready to assign it to anyone.

lindapaiste commented 10 months ago

We may also think about how we organize these lists of links. Personally I don't think that "Twitter" and "Instagram" make sense under the "New to p5.js?" heading. It makes more sense thematically to put them in some sort of "Community" list, alongside "Forum" and "Discord". We could also consider using icon buttons for social media channels.

The pink text on the version numbers makes them look like links, which they aren't. They could be links to the releases on GitHub which documents what was changed in each version.

lindapaiste commented 10 months ago

Mockups of ideas

  • Shrink the logo on very small screens. Maybe move the version numbers to the right of the logo?

With a smaller logo it's easier to get everything to fit without scrolling.

Screenshot 2023-12-09 16 15 39

  • Rethink the positioning on "medium" widths where we can fit two columns. Maybe a smaller logo up top and the two lists go next to each other below it?

I kept everything the same size but rearranged the elements in a way that makes more sense visually.

Screenshot 2023-12-09 15 18 38

I still need to play around with landscape mode. I think we'll want to look at both the height and the width when determining whether a modal is full-screen. Right now we only look at the width. So it's not giving us much room to work with. If we bump this up to full screen it can probably keep the same general layout as on desktop. Screenshot 2023-12-09 16 26 29

aryanas159 commented 10 months ago

@lindapaiste How's this for Mobile Landscape? image

Swarnendu0123 commented 10 months ago

Mockups of ideas

  • Shrink the logo on very small screens. Maybe move the version numbers to the right of the logo?

With a smaller logo it's easier to get everything to fit without scrolling.

Screenshot 2023-12-09 16 15 39

  • Rethink the positioning on "medium" widths where we can fit two columns. Maybe a smaller logo up top and the two lists go next to each other below it?

I kept everything the same size but rearranged the elements in a way that makes more sense visually.

Screenshot 2023-12-09 15 18 38

I still need to play around with landscape mode. I think we'll want to look at both the height and the width when determining whether a modal is full-screen. Right now we only look at the width. So it's not giving us much room to work with. If we bump this up to full screen it can probably keep the same general layout as on desktop. Screenshot 2023-12-09 16 26 29

Great idea @lindapaiste ! as a contributor and love for p5.js, can you please suggest how to proceed.

raclim commented 10 months ago

It makes more sense thematically to put them in some sort of "Community" list, alongside "Forum" and "Discord". We could also consider using icon buttons for social media channels.

I still need to play around with landscape mode. I think we'll want to look at both the height and the width when determining whether a modal is full-screen. Right now we only look at the width. So it's not giving us much room to work with. If we bump this up to full screen it can probably keep the same general layout as on desktop.

@lindapaiste I'm wondering if turning this into a dedicated page (like the ones in /account, /sketches, and /collections) might be better and give more space and freedom to organize the links?

The pink text on the version numbers makes them look like links, which they aren't. They could be links to the releases on GitHub which documents what was changed in each version.

Completely agree here!

aryanas159 commented 10 months ago

@lindapaiste Do you wanna go with the suggestions made by @raclim?

Akhilbisht798 commented 10 months ago

I agree with @raclim here. Instead of a modal which we are using we can make it a new page like account etc. And there we also put p5.js instagram, twitter and discord with icons and stuff for joining our community. It will provide much more space to experiment around.

And this page will act more like a gateway from where user can assess entire community of p5.js.

Kiran1504 commented 10 months ago

the landscape one is easy to be solved and a pr is created for the same. Mobile screen and logo and scrolling changes can be done if required in issue #2686

adityagarg06 commented 9 months ago

Hi @lindapaiste, Here is a proposed solution :

For bigger screens (1024px), languages with longer words, such as Italian, were overflowing, so changed the width and also changed the space between the footer items: 1

For screen width between (430px to 900px), Changing the size of the logo fixed the overflowing issue: 2

For screen width less than (430px), The flex-direction is changed to row. We can remove the scrolling by reducing the font size and logo size: 4