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

[*] FO : Update #PSCSX-5495 input label and input title enhancements #2983

Merged
merged 2 commits into from
May 11, 2015
Merged

[*] FO : Update #PSCSX-5495 input label and input title enhancements #2983

merged 2 commits into from
May 11, 2015

Conversation

inetbiz
Copy link
Contributor

@inetbiz inetbiz commented May 6, 2015

Line 317 does not have a for association for the label. Line 318 input title for (WCAG 2.0) hints and accessibility devices.

@inetbiz inetbiz changed the title [*] FO: PSCSX-5495 Fixes input label and input title [*] FO : PSCSX-5495 Fixes input label and input title May 6, 2015
@inetbiz inetbiz changed the title [*] FO : PSCSX-5495 Fixes input label and input title [*] FO : #PSCSX-5495 Fixes input label and input title May 6, 2015
@inetbiz inetbiz changed the title [*] FO : #PSCSX-5495 Fixes input label and input title [*] FO : Fix #PSCSX-5495 input label and input title enhancements May 6, 2015
@inetbiz inetbiz changed the title [*] FO : Fix #PSCSX-5495 input label and input title enhancements [*] FO : Update #PSCSX-5495 input label and input title enhancements May 6, 2015
@julienbourdeau julienbourdeau self-assigned this May 6, 2015
@julienbourdeau
Copy link
Contributor

Hi,

Thanks for your contribution.

The cursor on quantity field is I-beam type so the title render next to it, it's not very friendly for a most users. Even if we agree it's better for accessibility, we'd rather not add the title.

Please remove it and I'll be happy to merge your PR with for="quantity_wanted" addition.

Best regards,

@inetbiz
Copy link
Contributor Author

inetbiz commented May 6, 2015

All done @julienbourdeau This was much more of a learning experience for me. Learning the process to contribute.

@jnadaud jnadaud merged commit 92ac9be into PrestaShop:1.6 May 11, 2015
@inetbiz inetbiz deleted the PSCSX-5495 branch May 12, 2015 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants