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

Added Dropdown option , FIXES #367 #368

Merged
merged 3 commits into from
Apr 9, 2023

Conversation

Cshiva773
Copy link
Contributor

🛠️ Fixes Issue

Related Issue/Addition to code

  • Adding a Dropdown option in front of contributors' name

👨‍💻 Changes proposed

-Added Dropdown options in the contributor list to display their contributions in that dropdown list

Type of change

  • [✔️] New feature (non-breaking change which adds functionality)

✔️ Check List (Check all the applicable boxes)

  • [✔️] My code follows the code style of this project.
  • [✔️] This PR does not contain plagiarized content.
  • [✔️] The title of my pull request is a short description of the requested changes.

📷 Screenshots

open

@netlify
Copy link

netlify bot commented Apr 8, 2023

Deploy Preview for mogulchristmas ready!

Name Link
🔨 Latest commit 8bf153f
🔍 Latest deploy log https://app.netlify.com/sites/mogulchristmas/deploys/6432d2a5ad0c5d00082d54bb
😎 Deploy Preview https://deploy-preview-368--mogulchristmas.netlify.app/credits
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are accessibility issues in these changes.

<h3><a href="https://github.com/KendallDoesCoding/mogul-christmas/pull/291">Worked on the hyperlinks in JS for the "You May Like" directory for the "Pagination" update.</h3></a>

<div class="dropdown">
<a href="https://github.com/KendallDoesCoding"><h1>KendallDoesCoding<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.



<div class="dropdown">
<a href="https://github.com/TechStudent11"><h1>TechStudent10<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.



<div class="dropdown">
<a href="https://github.com/aliraza944"><h1>Ali Raza<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

</br>

<div class="dropdown">
<a href="https://github.com/JeffBucherDev"><h1>JeffBucherDev<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.



<div class="dropdown">
<a href="https://github.com/Bibhavshah"><h1>Bibhav Shah<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

</br>

<div class="dropdown">
<a href="https://github.com/Michael-Obele"><h1>Michael Obele<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

</br>

<div class="dropdown">
<a href="https://github.com/Weave-Trix"><h1>Weave-Trix<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

</br>

<div class="dropdown">
<a href="https://github.com/fk00750"><h1>F.K<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.



<div class="dropdown">
<a href="https://github.com/ArhyaSaha"><h1>Arhya Saha<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

</br>

<div class="dropdown">
<a href="https://github.com/frazie"><h1>Frazier Odhiambo<button class="dropbtn"><i class="arrow down"></i></h1></a></button>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

Copy link
Owner

@KendallDoesCoding KendallDoesCoding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • I want it on tap that the contributions display (the hover looks quite nice though, so keep that as well)
  • Can you work on the styling of the arrow just a little bit?

@Cshiva773
Copy link
Contributor Author

i will try my best

@Cshiva773
Copy link
Contributor Author

I have changed the styling of the arrow, please check that out

@KendallDoesCoding
Copy link
Owner

I want it on tap that the contributions display (the hover looks quite nice though, so keep that as well)

can you not do this?

Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 You fixed the issue(s)! Great work.

@KendallDoesCoding
Copy link
Owner

@Cshiva773 - Thanks for the contribution!
Please request to be assigned to a issue next time, before working on it.

@Cshiva773
Copy link
Contributor Author

Cshiva773 commented Apr 9, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[JS, CSS] Dropdown for contributions by contributors
2 participants