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

Check labelling for logging in and signing up users #4080

Closed
Angamanga opened this issue Oct 5, 2020 · 18 comments · Fixed by ushahidi/platform-client#1554
Closed

Check labelling for logging in and signing up users #4080

Angamanga opened this issue Oct 5, 2020 · 18 comments · Fixed by ushahidi/platform-client#1554
Labels
Codebase: Client Indicates issue work will be in client Hacktoberfest Issues open to folks participating in hacktoberfest Theme: accessibility
Milestone

Comments

@Angamanga
Copy link
Contributor

Angamanga commented Oct 5, 2020

There are two ways labels are handled in the platform, depending on the markup of the form:

<label for="username">Username</label>
<input id="username" type="text" />

and

<label>Username
     <input id="username" type="text" />
</label>

Go through the following forms and make sure the element is either using the for-attribute to associate the form-control or is encapsulating the form-control like the second example.

The login and sign-up forms are found when clicking on Login and Signup in the lefthand mode-bar:
Screenshot 2020-10-05 at 12 20 20

The forgot-password-form is found when clicking on "Forgot password" in the login-form:
Screenshot 2020-10-05 at 12 22 15

The files related to the above form is found here:

If no adjustments is needed, just comment in this issue that you have worked through the issues and checked, if adjustment is needed, make the necessary changes and create a PR 🎉 If changes are needed, make sure the form still looks the same as before and that is has no effect on the visual appearance of the form.

Aha! Link: https://ushahiditeam.aha.io/features/PROD-836

@Angamanga Angamanga added Codebase: Client Indicates issue work will be in client Community Task Issues that the Ushahidi OSS community is encouraged to contribute to first-timers-only I’m willing to hold your hand so you can make your first PR. This issue is a bit easier than normal. good-first-issue Issues that are ideal for first time contributors to OSS or first time contributing to the platform Theme: accessibility labels Oct 5, 2020
@Angamanga Angamanga added this to the Hacktoberfest milestone Oct 5, 2020
@Angamanga Angamanga moved this from Proposed to Ready for development in Hacktoberfest '20'*2 🏠🍺 Oct 5, 2020
@Angamanga Angamanga added Hacktoberfest Issues open to folks participating in hacktoberfest Outreachy labels Oct 5, 2020
@Akshansh99
Copy link

Can I work on this?

@Akshansh99
Copy link

I've checked all the mentioned files and no further adjustment is required.

@niksseif
Copy link

niksseif commented Oct 5, 2020

Hi there,
I went through all the files mentioned above and all the labels have for attribute same as the id of input field.

@Angamanga
Copy link
Contributor Author

@Akshansh99 @niksseif Thank you for your contribution 🎉 !

@Angamanga Angamanga moved this from Ready for development to Accepted in Hacktoberfest '20'*2 🏠🍺 Oct 6, 2020
@Angamanga Angamanga removed Community Task Issues that the Ushahidi OSS community is encouraged to contribute to Hacktoberfest Issues open to folks participating in hacktoberfest Outreachy first-timers-only I’m willing to hold your hand so you can make your first PR. This issue is a bit easier than normal. good-first-issue Issues that are ideal for first time contributors to OSS or first time contributing to the platform labels Oct 6, 2020
@Angamanga
Copy link
Contributor Author

@all-contributors please add @Akshansh99 for ally

@allcontributors
Copy link
Contributor

@Angamanga

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

@Angamanga
Copy link
Contributor Author

@all-contributors please add @Akshansh99 for a11y

@allcontributors
Copy link
Contributor

@Angamanga

I've put up a pull request to add @Akshansh99! 🎉

@Angamanga
Copy link
Contributor Author

@all-contributors please add @niksseif for a11y

@allcontributors
Copy link
Contributor

@Angamanga

I've put up a pull request to add @niksseif! 🎉

@error404-sp
Copy link

Hi @Angamanga ,
I came across this project from the outreachy website and found it interesting. I went through the given links and found that some minimal adjustments can be made to improve accessibility.
I will like to be assigned this issue.
Could you please provide the docs to set up environment if it's necessary in this project.

@Angamanga
Copy link
Contributor Author

@error404-sp
Copy link

Hi @Angamanga ,
I have created the changes in the above files. Kindly review it.
ushahidi/platform-client#1554 (comment)
I would really appreciate feedback on it.
I have refered to the W3C standards for the changes.
Thanks

@Angamanga
Copy link
Contributor Author

@error404-sp Thank you for your contribution! I will review shortly!

@Angamanga Angamanga moved this from Accepted to To review in Hacktoberfest '20'*2 🏠🍺 Oct 8, 2020
@Angamanga Angamanga added the Hacktoberfest Issues open to folks participating in hacktoberfest label Oct 8, 2020
@TunrayoIlawole
Copy link

Can I work on this?

@rowasc
Copy link
Contributor

rowasc commented Oct 8, 2020

@TunrayoIlawole Hey. There's already someone working on it, so in this case, no. However you can check the other tickets in the hacktoberfest and outreachy labels. Thank you!

@tuxpiper
Copy link
Member

tuxpiper commented Dec 2, 2020

@all-contributors please add @error404-sp for code

@allcontributors
Copy link
Contributor

@tuxpiper

I've put up a pull request to add @error404-sp! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Codebase: Client Indicates issue work will be in client Hacktoberfest Issues open to folks participating in hacktoberfest Theme: accessibility
Projects
Development

Successfully merging a pull request may close this issue.

7 participants