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

Adjust Option Orientation in CreateRadio() reference. #7366

Closed
1 of 17 tasks
perminder-17 opened this issue Nov 7, 2024 · 5 comments · Fixed by #7367
Closed
1 of 17 tasks

Adjust Option Orientation in CreateRadio() reference. #7366

perminder-17 opened this issue Nov 7, 2024 · 5 comments · Fixed by #7367

Comments

@perminder-17
Copy link
Contributor

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build process
  • Unit testing
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

No response

Web browser and version

No response

Operating system

No response

Steps to reproduce this

Steps:

  1. Go to link : https://p5js.org/reference/p5/createRadio/
  2. See the reference

In PR #7315 we noticed,

image

A proper alignment would be nice to have.

@devesh-2002
Copy link

Hi. May I take this issue?

@perminder-17
Copy link
Contributor Author

Hi. May I take this issue?

Sure, I'll assign you.

@Prajyot05
Copy link
Contributor

Prajyot05 commented Nov 8, 2024

@perminder-17 I have solved the issue, should I raise my PR? @devesh-2002 are you facing any issue?
Here's how it looks:
Screenshot 2024-11-08 at 3 26 57 PM

@devesh-2002
Copy link

Hi @Prajyot05 , you can take this issue if you have already worked on it.

@Prajyot05
Copy link
Contributor

@devesh-2002 alright, thanks

davepagurek added a commit that referenced this issue Nov 25, 2024
fixes issue #7366: Adjust Option Orientation in CreateRadio() reference.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants