Skip to content

Notation for the need of login to upload (image) files #3079

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

Open
prodka opened this issue Apr 12, 2024 · 5 comments · May be fixed by #3497
Open

Notation for the need of login to upload (image) files #3079

prodka opened this issue Apr 12, 2024 · 5 comments · May be fixed by #3497
Labels
Enhancement Improvement to an existing feature Good First Issue A beginner-friendly issue, great for first-time contributors Needs Design Requires design input before moving forward

Comments

@prodka
Copy link

prodka commented Apr 12, 2024

Increasing Access

This feature will allow people to navigate the use of image() and upload file feature easily. More documentation increases access.

Feature enhancement details

Recently, I was working on the p5 web editor without logging in. I was trying to play with image() and trying to upload an image from my computer. However, only after a while I realized I needed to login to have the option to upload.

Either within the web editor or the image() reference page, if there is an indicator that you need to login to use the upload file feature might be helpful.

@prodka prodka added the Enhancement Improvement to an existing feature label Apr 12, 2024
Copy link

welcome bot commented Apr 12, 2024

Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, be sure to follow the issue template if you haven't already.

@raclim raclim added the Needs Design Requires design input before moving forward label Apr 12, 2024
@Harshit-7373
Copy link
Contributor

@raclim @prodka Could you provide me with a design of how it should look? This issue needs further discussion.

@raclim raclim added the Good First Issue A beginner-friendly issue, great for first-time contributors label May 13, 2025
@yashodipmore
Copy link

Hi @raclim, @Harshit-7373 👋

I'd love to work on this issue. Could you please assign it to me?

💡 My Proposed Approach:
Add a visible message or tooltip in the UI near the image upload section that clearly states:
“You must be logged in to upload files.”

Update the image() function reference or upload modal with an alert or subtle badge (e.g., 🔒 Login required) to indicate the restriction upfront.

If needed, I’ll propose a minimal design (non-intrusive, accessible) using a simple div or span with conditional rendering based on user auth status.

I’ll make sure the messaging is accessible and fits within the current UI theme and layout.

Looking forward to contributing and improving accessibility for users! 😊

@raclim
Copy link
Collaborator

raclim commented May 16, 2025

@yashodipmore I think that sounds okay so far! If possible, it'd be great if you could provide a brief screenshot or wireframe visualizing what this could look like on the editor.

@yugalkaushik
Copy link
Contributor

Anyone working on this issue:

Since the upload file button is hidden when the user is not logged in, it would be helpful to show a clear message elsewhere to inform users that login is needed to upload images.
This message could be placed beside the Login and Sign Up buttons until the user is logged in, saying something like:
"Login required to upload images."
This way, users see the reminder right where they can take action, making it clear why the upload option isn’t available yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement to an existing feature Good First Issue A beginner-friendly issue, great for first-time contributors Needs Design Requires design input before moving forward
Projects
None yet
5 participants