GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — [Spec] Relax user activation requirement for show() by nickburris · Pull Request #1009 · w3c/payment-request

Conversation

@nickburris

@nickburris nickburris commented Jun 12, 2023 •
edited by pr-preview Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

This gives the user agent the ability to relax the user activation requirement on the PaymentRequest.show() method.

This spec change largely follows the relevant change outlined in Secure Payment Confirmation: w3c/secure-payment-confirmation#236


Preview | Diff

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Nick Burris added 2 commits June 12, 2023 12:54
Nick Burris
[Spec] Relax user activation requirement for show()
8d1bf9e
Nick Burris
Fix markup
7cfb833
@nickburris

nickburris commented Jun 12, 2023

Copy link
Copy Markdown
Member Author

@rsolomakhin, PTAL for a first look! First time in this spec 😄

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

rsolomakhin
rsolomakhin previously requested changes Jun 12, 2023
View reviewed changes

@rsolomakhin rsolomakhin left a comment

Copy link
Copy Markdown
Collaborator

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Welcome to the spec! :-D

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread index.html Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread index.html Resolved

Uh oh!

There was an error while loading. Please reload this page.

@stephenmcgruer
stephenmcgruer requested a review from ianbjacobs June 19, 2023 12:59
ianbjacobs
ianbjacobs approved these changes Jun 19, 2023
View reviewed changes
@stephenmcgruer
stephenmcgruer marked this pull request as ready for review June 19, 2023 19:51
@ianbjacobs

ianbjacobs commented Jun 22, 2023

Copy link
Copy Markdown
Collaborator

Discussed today at the WPWG meeting: https://www.w3.org/2023/06/22-wpwg-minutes#t04

There were no objections to merging the pull request.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Nick Burris
Address comments
3f335a0
@nickburris
nickburris requested a review from rsolomakhin June 27, 2023 18:55
@stephenmcgruer
stephenmcgruer dismissed rsolomakhin’s stale review June 28, 2023 19:48

Issue addressed

@stephenmcgruer
stephenmcgruer self-requested a review June 28, 2023 19:48
stephenmcgruer
stephenmcgruer approved these changes Jun 28, 2023
View reviewed changes
@stephenmcgruer
stephenmcgruer merged commit d9352e2 into w3c:gh-pages Jun 28, 2023
@marcoscaceres

marcoscaceres commented Feb 5, 2024

Copy link
Copy Markdown
Member

It would have been good to also document and specify the redirect feature... I have concerns this went in without WebKit review because I don't think we support this model (and the redirect without requiring user activation... is that is that thing, like, in HTML?).

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@marcoscaceres

marcoscaceres commented Feb 5, 2024

Copy link
Copy Markdown
Member

also, please kindly always make us the PR template. Otherwise, we miss tests, and bugs filed on other browser engines.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@stephenmcgruer

stephenmcgruer commented Feb 7, 2024

Copy link
Copy Markdown
Collaborator

Hey Marcos; apologies for missing on the PR template, and I agree that we could have done better here to get WebKit views. I will push back slightly and note that at time of PR creation, to my knowledge Apple was not a member of the WPWG and so was not engaging on PaymentRequest/etc issues, however we could still have requested a standards position at the very least.

We deliberately wrote the spec change so that it was optional for the user agent to allow, but ack that it may not fully address your concerns. Happy to discuss in a future WPWG meeting!

and the redirect without requiring user activation... is that is that thing, like, in HTML?

I may not understand your question correctly, but one can redirect in HTML without any user interaction or activation. Via either JavaScript or 302 redirects. One generally cannot open pop-ups without user interaction (though I don't think it's actually spec'd - it's about the individual browsers pop-up blocker logic), so it's fair to note that this change is risky in that direction. We did weight that up for Chrome very seriously and implemented some mitigations, but ultimately felt the use-case of redirect from aggregator --> frictionless merchant checkout flow was worth supporting.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@stephenmcgruer stephenmcgruer mentioned this pull request Feb 13, 2024
Consider spec-ing when a user agent should allow activationless PaymentRequest #1022
Open
@ianbjacobs ianbjacobs mentioned this pull request Aug 1, 2024
CR Snapshot and CR Draft Update Request for Payment Request API - payment-request w3c/transitions#643
Closed
stephenmcgruer added a commit that referenced this pull request Aug 21, 2024
@stephenmcgruer
[Spec] Restore changes since REC
94ad189
As per the plan outlined in #1021#issuecomment-2107832273, we have now
created the CR Snapshot and published a new CR Draft
(https://www.w3.org/TR/payment-request/) that includes restoring
the address changes. The only remaining work for that plan is to restore
the previous changes that had been made since REC.

Changes included:
  - [Spec] Relax user activation requirement for show() #1009

Additional edits on tops of above changes:
  - Add note linking to issue #1022 for discussion of user activation
    relaxing conditions

Fixes #1025
@stephenmcgruer stephenmcgruer mentioned this pull request Aug 21, 2024
[Spec] Restore changes since REC #1029
Merged
stephenmcgruer added a commit that referenced this pull request Aug 21, 2024
@stephenmcgruer
[Spec] Restore changes since REC
c7c04d2
As per the plan outlined in #1021#issuecomment-2107832273, we have now
created the CR Snapshot and published a new CR Draft
(https://www.w3.org/TR/payment-request/) that includes restoring
the address changes. The only remaining work for that plan is to restore
the previous changes that had been made since REC.

Changes included:
  - [Spec] Relax user activation requirement for show() #1009

Additional edits on tops of above changes:
  - Add note linking to issue #1022 for discussion of user activation
    relaxing conditions

Fixes #1025
ianbjacobs pushed a commit that referenced this pull request Sep 9, 2024
@stephenmcgruer
[Spec] Restore changes since REC (#1029)
be6feef
As per the plan outlined in #1021#issuecomment-2107832273, we have now
created the CR Snapshot and published a new CR Draft
(https://www.w3.org/TR/payment-request/) that includes restoring
the address changes. The only remaining work for that plan is to restore
the previous changes that had been made since REC.

Changes included:
  - [Spec] Relax user activation requirement for show() #1009

Additional edits on tops of above changes:
  - Add note linking to issue #1022 for discussion of user activation
    relaxing conditions

Fixes #1025
This was referenced Mar 26, 2026
Relax transient activation requirement for DC API requests w3c-fedid/digital-credentials#486
Open
Revert activation relaxation in show() -- find a general platform solution instead #1064
Open
marcoscaceres added a commit that referenced this pull request Mar 27, 2026
@marcoscaceres
fix: restore transient activation requirement in show()
c0052f3
Reverts the MAY introduced in #1009 and removes the non-normative
"User activation requirement" section that blessed the workaround.

Adds a note pointing to #1064 where the platform-wide sanctioned
continuation problem is being tracked.

Closes #1064
This was referenced Mar 27, 2026
fix: restore transient activation requirement in show() #1066
Open
Question: Capability Delegation stalled -- specs are implementing local workarounds w3ctag/design-reviews#1213
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stephenmcgruer stephenmcgruer stephenmcgruer approved these changes
@ianbjacobs ianbjacobs ianbjacobs approved these changes
@rsolomakhin rsolomakhin Awaiting requested review from rsolomakhin rsolomakhin is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

5 participants

@nickburris @ianbjacobs @marcoscaceres @stephenmcgruer @rsolomakhin
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.