Skip to content

Conversation

himorin
Copy link
Contributor

@himorin himorin commented Sep 22, 2025

for part of #188 (1st one)

This pull request updates the geolocation specification's "Request a position" algorithm in index.html to clarify when steps should be performed in parallel, specifically during the permissions check and subsequent actions. The main change is to correct the algorithm's structure to reflect the intended parallel execution, improving clarity and aligning with implementation expectations.

Algorithm correction for permissions check:

  • Added an aside explaining the correction: the algorithm should only switch to "in parallel" when performing the permissions check and subsequent steps, not before.
  • Removed the previous sequential steps for requesting permission and handling denied permissions, and replaced them with a parallel execution block for these actions. [1] [2]
  • no spec text change, purely markup fix, using candidate correction
  • text in line 674 need to be reviewed
  • since fix spans over multiple li, candidate correction box are inserted at beginning of listing part
  • note, original (by means of "show current") had markup error, so html source diff will list some difference

Preview | Diff

@himorin
Copy link
Contributor Author

himorin commented Sep 22, 2025

ah, del/ins cannot be under ol,,, hrm.

@marcoscaceres
Copy link
Member

Markup is still invalid tho.... we might need to do nasty things... like add <li><del><ol> in there temporarily.

@marcoscaceres
Copy link
Member

I think we want this instead: #200

Copy link
Member

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should just go with #200 and not stress about it.

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.

3 participants