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

HTMLMapmlViewerElement lat, lon, zoom WebIDL attributes' description needs update #121

Open
prushforth opened this issue May 18, 2023 · 0 comments

Comments

@prushforth
Copy link
Member

prushforth commented May 18, 2023

The documentation for lat, lon and zoom, at least in the table, is slightly unclear. It says (in all three cases):

"Get or set the map's (latitude). Reflects to the (lat) content attribute. No effect on map dynamic state."

This should be amended to

"Get or set the map's (latitude) property and attribute. Changing this value has no effect on the map viewport. Such changes can be achieved with the zoomTo() method."

The use of "reflect" is not quite right, according to the HTML spec. In that spec "reflect" seems to mean that content attribute changes reflect to the WebIDL attribute, but the reverse, that is changing the content attribute because the WebIDL attribute changes is described as "changing", not reflecting:

In general this means that the IDL attribute getter returns the current value of the content attribute, and the setter changes the value of the content attribute to the given value.

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

No branches or pull requests

1 participant