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

Fixes #1263 - Fixes aria attributes typo in templates #1302

Merged
merged 1 commit into from
Jan 26, 2017

Conversation

MDTsai
Copy link
Contributor

@MDTsai MDTsai commented Jan 25, 2017

Fix 2 typo from ara-hidden -> aria-hidden. @karlcow , could you please help to check this?

Copy link
Member

@karlcow karlcow left a comment

Choose a reason for hiding this comment

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

@MDTsai

  1. could you change the commit message so it respects the syntax of the guidelines.
    https://github.com/webcompat/webcompat.com/blob/master/CONTRIBUTING.md#submission-guidelines

for example "Issue #1263 - fixes aria typo in template"

  1. Check also how we do titles for Pull Request.

+1 for the changes once this is modified.

@karlcow karlcow changed the title Issue 1263 - Fix typo Fixes #1263 - Fixes aria attributes typo in templates Jan 25, 2017
@karlcow
Copy link
Member

karlcow commented Jan 25, 2017

I modified the title of the PR according to our guidelines. :)

@karlcow
Copy link
Member

karlcow commented Jan 25, 2017

And thanks for the Pull Request.

Fix 2 typo from ara-hidden -> aria-hidden
@magsout
Copy link
Member

magsout commented Jan 25, 2017

thanks @MDTsai 👍

@karlcow
Copy link
Member

karlcow commented Jan 26, 2017

Perfect!

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