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

In `About` modal position of "Contribute" and "Report a bug" is not consistent

Open PiyushChandra17 opened this issue 2 years ago • 18 comments

p5.js version

2.8.0

What is your operating system?

Mac OS

Web browser and version

Chrome and 117.0.5938.132

Actual Behavior

In About modal position of "Contribute" and "Report a bug" is not consistent

p5-about

Expected Behavior

In About modal position of "Contribute" and "Report a bug" should be consistent

Steps to reproduce

Steps:

  1. Open the editor
  2. Go to Help dropdown
  3. Click on about
  4. Notice at the end of modal, position of contribute and report a bug is not consistent

PiyushChandra17 avatar Oct 24 '23 08:10 PiyushChandra17

Good catch. We recently added some responsive breakpoints to that component which are better than nothing but not perfect. I bet if you look through the git history of the _about.scss you can see what we used to have. I think there was previously a large left margin on the footer section to align it? We would want to put that back in, but only on large/desktop screens.

lindapaiste avatar Oct 24 '23 14:10 lindapaiste

Hello, is the expected design something like this. Aligned vertically with above columns image

bikrantjajware avatar Oct 25 '23 04:10 bikrantjajware

@PiyushChandra17 hello, is this the desired output, the Contribute and report links alligned with the two existing columns? If yes, I would like to create a pr. Screenshot_20231026_0438291

Ahmed-Armaan avatar Oct 25 '23 23:10 Ahmed-Armaan

@lindapaiste As I can see from history there was a larger left padding at the ".about__footer" so putting that back it would look something like this. Is this what you wanted it to be ?

Screenshot 2023-10-27 at 12 04 49 PM

And also "New to p5.js?" and "Resources" which are column title are also not align to column list, so is that ok then ignore it, I just removed there left padding it looks better for me, if you can comment on them as well.

Swapnil-2502 avatar Oct 27 '23 06:10 Swapnil-2502

Greetings, is this the desired result? If so, let me know so I can make a PR. image If not, please specify how the content should be styled.

mhsh312 avatar Oct 28 '23 18:10 mhsh312

I'd like to work on this issue can you assign it to me?

PiyushThapaa avatar Nov 06 '23 14:11 PiyushThapaa

@PiyushThapaa I am already working on it !!!!

yashpandey06 avatar Nov 06 '23 20:11 yashpandey06

@PiyushThapaa but if you have already done it you can make the pr

yashpandey06 avatar Nov 06 '23 20:11 yashpandey06

@PiyushThapaa but if you have already done it you can make the pr

no i haven't can you please help me i enable to host the server do you how to host the server locally after cloning

PiyushThapaa avatar Nov 07 '23 13:11 PiyushThapaa

you can simply use docker setup to start a localhost ...go through the contributing docs

yashpandey06 avatar Nov 13 '23 19:11 yashpandey06

@PiyushChandra17 I am going to raise pr for this particular issue soon ...but firstly let me know if you are working on this issue.

yashpandey06 avatar Nov 13 '23 19:11 yashpandey06

I can host the application locally using docker but when I make changes in the code it is not getting refreshed in the code. So I have to build a new image every time there is something I am doing wrong. Kindly help me with this.

udittripathi avatar Nov 20 '23 05:11 udittripathi

Is this still open? Can i work on it?

shree6 avatar Nov 25 '23 07:11 shree6

@udittripathi Same problem has been there for me.

shree6 avatar Nov 25 '23 10:11 shree6

@shree6 If you find the solution please le me know also

udittripathi avatar Nov 25 '23 10:11 udittripathi

@lindapaiste can u help me with this issue?

udittripathi avatar Nov 27 '23 11:11 udittripathi

Screenshot from 2023-12-18 15-30-15

Saikat123Dev avatar Dec 18 '23 11:12 Saikat123Dev

@lindapaiste If the issue is still not resolved, can I work on it?

shamvrueth avatar Jan 22 '24 14:01 shamvrueth

Closing to organize and conslidate issues, linked in Megathread.

raclim avatar Aug 13 '24 16:08 raclim