Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Changing default zoom to below 100% changes UI Scale to smaller but doesn't show in the setting #9334

Closed
srirambv opened this issue Jun 8, 2017 · 12 comments
Labels
accessibility feature/about-pages needs-info Another team member needs information from the PR/issue opener. stale usability

Comments

@srirambv
Copy link
Collaborator

srirambv commented Jun 8, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Changing default zoom to below 100% changes UI Scale to smaller but doesn't show in the setting

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Found on Windows 10 x64. Needs verification on macOS (cc: @alexwykoff @LaurenWags @luixxiul )

  • Brave Version (revision SHA):
    Brave 0.16.4
    rev 15d8d36
    Muon 4.0.0

  • Steps to reproduce:

    1. Clean install 0.16.4
    2. Enable Always show URL bar from general settings
    3. Set Default zoom to 90%, visually the UI gets smaller including the URLbar and buttons
    4. Change UI Scale to smaller, No visible changes to URL bar in terms of size
    5. Change UI Scale back to Normal, URL bar size is changed to default
    6. Change UI Scale back to smaller , visible URL bar size change
  • Actual result:
    Changing default zoom to below 100% changes UI Scale to smaller but doesn't show in the setting

  • Expected result:
    If default zoom is set to below 100% and UI scale is auto changed then the setting has to indicate it

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    ui

  • Any related issues:
    cc: @bsclifton

@luixxiul
Copy link
Contributor

luixxiul commented Jun 8, 2017

Absolutely that's the regression. Changing the value to 500% makes the browser completely unusable until clearing the profile.

#8144

The bug can be reproduced on 0.15.312 (d4f169a).

@luixxiul luixxiul modified the milestones: 0.17.x (Frozen, only critical adds from here), 0.18.x Jun 8, 2017
@luixxiul
Copy link
Contributor

luixxiul commented Jun 8, 2017

I put the issue to 0.17.x for now as it can be reproduced on the released version.

@srirambv
Copy link
Collaborator Author

The problem is the default zoom is changing the UI zoom level and not just the page zoom. So first you have to set the UI zoom level and then accordingly change the default page zoom level to adjust to need.

cc: @bsclifton

@alexwykoff alexwykoff removed this from the 0.17.x (Frozen, only critical adds from here) milestone Jun 12, 2017
@luixxiul luixxiul removed the settings label Jun 27, 2017
@luixxiul
Copy link
Contributor

On Debian:

clipboard01

I believe this is a regression. Is it not? @bsclifton

@bsclifton
Copy link
Member

@luixxiul what is the issue exactly? (sorry, I'm not sure if the problem is clearly stated?)

In your screenshot, if you didn't have the dev tools open, you'd be able to see the setting if you had the dev-console closed

@luixxiul
Copy link
Contributor

I cannot remember why I pinged you but this is the issue anyway:

bug

@cndouglas
Copy link

Confirmed with Brave 0.17.19 (4e46480) on macOS 10.12.x.

@bsclifton
Copy link
Member

bsclifton commented Jul 22, 2017

I don't think this is a bug, per-se... but we should have an easy way to go back.

Maybe if someone pics an extreme setting like this, we should make the change and then open a confirm dialog... "Did you mean to make this change?". We could even have the setting revert itself within 10 seconds if the person doesn't respond.

cc: @bradleyrichter @alexwykoff for thoughts and comment

@luixxiul
Copy link
Contributor

@bsclifton I have thought that UI scale can be changed via the setting toolbar and UI elements scale on about:preferences#advanced only like this.

bug

While the issue on above #9334 (comment) is that UI scale is changed via default zoom level, which is the same issue as reported here: #8144

@luixxiul luixxiul added the needs-info Another team member needs information from the PR/issue opener. label Jul 22, 2017
@cndouglas
Copy link

This is a bug because the zoom level should not be affecting the browser UI.

@luixxiul
Copy link
Contributor

The issue above #9334 (comment) is no longer reproduced on 0.18.12

@cndouglas
Copy link

Still reproducible with Brave 0.20.30 on macOS 10.13.x.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility feature/about-pages needs-info Another team member needs information from the PR/issue opener. stale usability
Projects
None yet
Development

No branches or pull requests

5 participants