jjwilliams42 / generator-aspnet-semanticui

Yeoman ASP.NET Generator with Semantic UI Library
1 stars 2 forks source link

Add text context to toggles and i-con only links/controls #7

Open peterblazejewicz opened 8 years ago

peterblazejewicz commented 8 years ago

The markup is missing some text context that is already added to aspnet/Templates (to be a good, web citizen). They originally missed this as well: https://github.com/aspnet/Templates/commit/2a8fc46db1656064145a251512c28e2ca917e5c7 The Semantic UI should have something similar for screen readers, if not, it'd be nice to provide an util class for this, add to site.css and just use it in markup. Otherwise current toggles are just clickable, because some people just see they are on the screen. This is not bad markup, but templates should do better.