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

Correction of the slider width to no longer display the horizontal scroll + doc for deviceType.getByCategory #414

Merged
merged 4 commits into from
Sep 20, 2018

Conversation

piznel
Copy link
Contributor

@piznel piznel commented Sep 10, 2018

If you set slider-horizontal-width to 100%; the slider-tick-label goes beyond the container at max value :
before:
enter image description here
After:
enter image description here

…rolls

 If you set `slider-horizontal-width` to `100%`; the slider-tick-label goes beyond the container at max value :
**before:**
![enter image description here](https://raw.githubusercontent.com/piznel/img/master/Gladys-slider_before.png)
 **After:**
![enter image description here](https://raw.githubusercontent.com/piznel/img/master/Gladys-slider_after.png)
@piznel piznel changed the title Correction of the slider width to no longer display the horizontal scroll Correction of the slider width to no longer display the horizontal scroll + doc for deviceType.getByCategory Sep 18, 2018
@Pierre-Gilles
Copy link
Contributor

Pierre-Gilles commented Sep 18, 2018

Hey!

Thanks for the PR!

I'm trying to reproduce your bug but i can't manage to reproduce it...

When exactly does it happens? Is there with a specific screen resolution? :)

@piznel
Copy link
Contributor Author

piznel commented Sep 18, 2018

I have it with Windows 7, on Chrome and Firefox (latest version), at the resolution recommended by my screen.
On Ipad too :
3c50b6ef-5716-41f5-ba2e-7c813685b440

@Pierre-Gilles
Copy link
Contributor

Got it! I see what you mean and i'm able to reproduce indeed!

I'm merging right now

@Pierre-Gilles Pierre-Gilles merged commit f1bc855 into GladysAssistant:master Sep 20, 2018
R6n0 pushed a commit to R6n0/Gladys that referenced this pull request Dec 2, 2020
…roll + doc for deviceType.getByCategory (GladysAssistant#414)

* Correction of the slider width to no longer display the horizontal scrolls

 If you set `slider-horizontal-width` to `100%`; the slider-tick-label goes beyond the container at max value :
**before:**
![enter image description here](https://raw.githubusercontent.com/piznel/img/master/Gladys-slider_before.png)
 **After:**
![enter image description here](https://raw.githubusercontent.com/piznel/img/master/Gladys-slider_after.png)

* Add deviceType.getByCategory doc

* Add 'gladys.deviceType.getByCategory' to jsdoc.js
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.

2 participants