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

V8: Move misplaced colon in language selector #6692

Merged
merged 3 commits into from
Oct 15, 2019

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Oct 15, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

Description

As an unintended side effect of #6538, preceding colons have appeared in the language selector:

image

The colons are intended for screen readers, so this PR moves these colons inside the sr-only block, and thus the language selector looks normal again:

image

Copy link
Contributor

@dawoe dawoe left a comment

Choose a reason for hiding this comment

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

Hi @kjac

Thanks for this tremendous effort. Looks good to me 👍

Dave Community Pull Request team

@nul800sebastiaan nul800sebastiaan merged commit 178185e into umbraco:v8/dev Oct 15, 2019
@nul800sebastiaan
Copy link
Member

Ha! I just fixed this one before I saw this.. 4e345a8

This fix seems even better, so I'll merge it on top of that. Thanks @kjac!

@kjac kjac deleted the v8/fix/language-selector-label branch October 15, 2019 13:51
emmaburstow pushed a commit that referenced this pull request Oct 27, 2019
…ton (#6561)

* Improves accessability of the three dot tree options button

* Updated list view headers styling to have the link/hand cursor visible only if the header is sortable (and hence clicking has an effect)

* Allows members to be ordered by additional system fields (#6575)

* Fix ContentType.Alias matching in PublishedContentExtensions an… (#6577)

* Fixes an error in #6538 by moving the colon character to the sr-only span

* V8: Move misplaced colon in language selector (#6692)

* V8: It should be possible to disallow all types at content root (#6580)

* Pick macro parameters in an infinite editor like content type properties (#6586)

* Reload node children after publishing with descendants

* Correct URL decoding of macro partial view names (#6592)

* Fix semantics for list views (#6595)

* fixes test

* fixing tests

* Improved menu context
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants