p5.js-web-editor icon indicating copy to clipboard operation
p5.js-web-editor copied to clipboard

Megathread: issues with "About" modal responsive design

Open lindapaiste opened this issue 2 years ago • 12 comments

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:

  • [x] #2700
  • [x] #2531
  • [x] #2565
  • [x] #2538

Proposed solutions:

  • Set explicit column widths (33.3%, 50%, 100%) for various breakpoints using media queries.
  • Rearrange the HTML so that the about__footer is not a sibling of the columns.
  • Shrink the logo on very small screens. Maybe move the version numbers to the right of the logo?
  • 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?

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.

lindapaiste avatar Dec 08 '23 22:12 lindapaiste

@lindapaiste Can I be assigned to this issue?

aryanas159 avatar Dec 09 '23 05:12 aryanas159

image How does this layout look for a mobile breakpoint?

aryanas159 avatar Dec 09 '23 05:12 aryanas159

@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 avatar Dec 09 '23 06:12 lindapaiste

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 avatar Dec 09 '23 20:12 lindapaiste

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

lindapaiste avatar Dec 09 '23 22:12 lindapaiste

@lindapaiste How's this for Mobile Landscape? image

aryanas159 avatar Dec 10 '23 05:12 aryanas159

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.

swrno avatar Dec 11 '23 06:12 swrno

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!

raclim avatar Dec 12 '23 23:12 raclim

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

aryanas159 avatar Dec 14 '23 07:12 aryanas159

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.

Akhilbisht798 avatar Dec 14 '23 07:12 Akhilbisht798

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

Kiran1504 avatar Dec 14 '23 09:12 Kiran1504

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

adityagarg06 avatar Dec 16 '23 14:12 adityagarg06