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

fix: add role attribute and visually hidden text #1015

Merged
merged 4 commits into from
Sep 28, 2020
Merged

Conversation

cidermitaina
Copy link
Member

@cidermitaina cidermitaina commented Sep 23, 2020

Related URL

https://smarthr.atlassian.net/browse/SHRUI-151

Overview

Loaderが達成基準 1.1.1 非テキストコンテンツに非適合なので修正しました。

What I did

  • Loader の Wrapper に role=status を追加
  • 子要素に visually-hidden なテキストを追加

@cidermitaina cidermitaina requested a review from a team as a code owner September 23, 2020 08:39
@cidermitaina cidermitaina self-assigned this Sep 23, 2020
@cidermitaina cidermitaina requested review from diescake and im36-123 and removed request for a team September 23, 2020 08:39
@reg-suit
Copy link

reg-suit bot commented Sep 23, 2020

✨✨ That's perfect, there is no visual difference! ✨✨

Check out the report here.

@netlify
Copy link

netlify bot commented Sep 23, 2020

Deploy preview for smarthr-ui ready!

Built with commit b78a7c4

https://deploy-preview-1015--smarthr-ui.netlify.app

@cidermitaina
Copy link
Member Author

@im36-123 @diescake
Please review!

@cidermitaina cidermitaina changed the title [WIP] fix: add role attribute and visually hidden text fix: add role attribute and visually hidden text Sep 24, 2020
im36-123
im36-123 previously approved these changes Sep 25, 2020
Copy link
Contributor

@im36-123 im36-123 left a comment

Choose a reason for hiding this comment

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

Tank you for your contribution !!!
LGTM 🎉

@wmoai
Copy link
Contributor

wmoai commented Sep 25, 2020

横から失礼します!:bow:
styleの関係で一番右のバーが動かなくなってるみたいです!
out

@cidermitaina
Copy link
Member Author

@wmoai
ほんとだ!気付かなかったです…!🙇
ありがとうございます😊

こちらで修正しましたので、ご確認お願いします!
e19239b

Copy link
Contributor

@wmoai wmoai left a comment

Choose a reason for hiding this comment

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

LGTM!

@cidermitaina cidermitaina merged commit 536e304 into master Sep 28, 2020
@cidermitaina cidermitaina deleted the SHRUI-151 branch September 28, 2020 05:16
wmoai pushed a commit that referenced this pull request Oct 2, 2020
* fix: add role attribute and visually hidden text

* fix: move visually hidden text in loader
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.

3 participants