Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

paper-slider only respects [dir=rtl] but not direction: ltr; in CSS nor nested [dir] #190

Description

@danbeam

Description

paper-slider added RTL support via :host-context([dir="rtl"]) in #181

This is super, except there are a few ways of changing text direction that it doesn't detect.

For example:

paper-slider {
  direction: rtl;
}

or

<html dir="rtl">
<paper-slider dir="ltr">

Expected outcome

Slider's direction and effective text direction agree.

Actual outcome

In some cases, the slider direction differs from the text direction.

Live Demo

https://jsfiddle.net/7qxd7cjx/

Steps to reproduce

  1. plop <paper-slider style="direction: rtl;"> or <div dir=rtl><paper-slider dir=ltr> onto a page
  2. slide the knob

Browsers Affected

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 11
  • IE 10

(Note: I assume this applies to all browsers but haven't actually checked).

/cc @dschuyler @keanulee

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions