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

window.outerHeight on Mobile #17

Closed
bramus opened this issue Aug 23, 2022 · 4 comments
Closed

window.outerHeight on Mobile #17

bramus opened this issue Aug 23, 2022 · 4 comments
Labels
Browser Bug To be followed up at the browser vendor Discussed Misc Spec Issue To be followed up at proper venue that specced this

Comments

@bramus
Copy link
Collaborator

bramus commented Aug 23, 2022

window.outerHeight is specced as follows:

The outerHeight attribute must return the height of the client window. If there is no client window this attribute must return zero.

There’s a few interesting things going on with on Mobile, as found through testing

@bramus
Copy link
Collaborator Author

bramus commented Sep 7, 2022

Discussed during #20.

Looks like a bug in browsers,. Additionally, the term “client window” in the definition is not defined by itself.

There’s an existing bug for Firefox on Android: https://bugzilla.mozilla.org/show_bug.cgi?id=712225

Emilio notes that the value is sometimes spoofed for security reasons.

API also seems low-prio / not used often.

@bramus bramus added Discussed Spec Issue To be followed up at proper venue that specced this Browser Bug To be followed up at the browser vendor labels Sep 7, 2022
@bramus
Copy link
Collaborator Author

bramus commented Oct 5, 2022

  • @dcrousso Can we discuss again? What did we resolve on?
  • @emilio Spec needs update
  • @bramus What about the spoofed value?
  • @emilio Tor Browser defo wants this. We might not need to account for that in the spec
  • @bramus Or maybe a small line that says UAs might not return a proper value for tracking reasons
  • @skobes What about screen size?
  • @emilio Tor also spoofs that.
  • @skobes Does the spec have about that?
  • @emilio Don’t think so.
  • @emilio Sometimes they (screen, inner, outer) are all the same – e.g. Tor
  • @emilio A small note in the spec would be fine

@bramus
Copy link
Collaborator Author

bramus commented Oct 5, 2022

** cssom-view spec

@bramus
Copy link
Collaborator Author

bramus commented Dec 16, 2022

Issues have been filed at the CSS WG.

@bramus bramus closed this as completed Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Browser Bug To be followed up at the browser vendor Discussed Misc Spec Issue To be followed up at proper venue that specced this
Projects
None yet
Development

No branches or pull requests

1 participant