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

Inconsistent slider button sizes #420

Closed
KatieWoe opened this issue Aug 2, 2019 · 6 comments
Closed

Inconsistent slider button sizes #420

KatieWoe opened this issue Aug 2, 2019 · 6 comments

Comments

@KatieWoe
Copy link
Contributor

KatieWoe commented Aug 2, 2019

Test device
Dell
Operating System
Win 10
Browser
chrome
Problem description
For phetsims/qa#389
The arrows that adjust the sliders become small on the diameter slider, but not the eccentricity slider, when doubling strings on the Diffraction screen. Smaller buttons seem to be what occur in other parts of the sim.

Visuals
buttonsizes

Troubleshooting information:

!!!!! DO NOT EDIT !!!!!
Name: ‪Wave Interference‬:‪Wave Interference‬
URL: https://phet-dev.colorado.edu/html/wave-interference/2.0.0-dev.7/phet/wave-interference_en_phet.html?stringTest=double
Version: 2.0.0-dev.7 2019-07-30 22:17:13 UTC
Features missing: touch
Flags: pixelRatioScaling
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.142 Safari/537.36
Language: en-US
Window: 1536x722
Pixel Ratio: 2.5/1
WebGL: WebGL 1.0 (OpenGL ES 2.0 Chromium)
GLSL: WebGL GLSL ES 1.0 (OpenGL ES GLSL ES 1.0 Chromium)
Vendor: WebKit (WebKit WebGL)
Vertex: attribs: 16 varying: 30 uniform: 4096
Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32)
Max viewport: 16384x16384
OES_texture_float: true
Dependencies JSON: {}

@arouinfar
Copy link
Contributor

@samreid is it possible to always maintain tweaker size, like the Eccentricity slider? The small tweaker buttons seem a bit problematic.

@arouinfar arouinfar removed their assignment Aug 2, 2019
@samreid
Copy link
Member

samreid commented Aug 3, 2019

Good catch @KatieWoe. The problem was introduced by phetsims/scenery-phet#513 where we set the tweaker button size to match the size of the number display. In this case, the large text has made the number display shorter, so the arrow buttons have shrunken too. This problem will impact every sim with NumberControl. Not sure what's the best solution--for now I'll reopen phetsims/scenery-phet#513 for discussion.

@KatieWoe
Copy link
Contributor Author

KatieWoe commented Aug 5, 2019

Probably moot at this point, but it does happen to a degree even without the string tests:
sizeisdifferent

@samreid
Copy link
Member

samreid commented Aug 6, 2019

I pushed a proposed fix in phetsims/scenery-phet#513 and it's up for review.

@samreid
Copy link
Member

samreid commented Aug 7, 2019

The fix in phetsims/scenery-phet#513 has been code reviewed. @KatieWoe can you please test it in master? Close if all is well. Also note #422 which we will handle separately.

@KatieWoe
Copy link
Contributor Author

KatieWoe commented Aug 7, 2019

Looks good in master

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

No branches or pull requests

3 participants