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

Content attr as a variable, aria-label as default, and :focus state for better accessibility #56

Closed
wants to merge 4 commits into from

Conversation

maximelebreton
Copy link

In response to that issue: #19

This pull request needs documentation, and build.

@kazzkiq kazzkiq added this to the v0.5.0 milestone Mar 1, 2017
@kazzkiq
Copy link
Owner

kazzkiq commented Aug 14, 2017

This is a breaking change. Will be pushing it to a major release.

@kazzkiq kazzkiq removed this from the v0.5.0 milestone Aug 14, 2017
@gu-stav
Copy link

gu-stav commented Oct 11, 2017

@kazzkiq Could you give an estimation when you are planning the 0.5.0 release?

@kazzkiq kazzkiq mentioned this pull request Oct 16, 2017
@fractaledmind fractaledmind mentioned this pull request Dec 22, 2017
@kazzkiq
Copy link
Owner

kazzkiq commented Jun 14, 2019

I've worked on lots of updates in the lib for an upcoming v1.0, and most of the fixes in this PR were addressed there.

@maximelebreton thanks a lot for putting effort on this PR, but since other things also changed for v1.0, this can no longer be merged.

Also, sorry for the delay to take a closer look at this.

@kazzkiq kazzkiq closed this Jun 14, 2019
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