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.

Sign up for GitHub

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

Jump to bottom

Navigator: cookieEnabled warning regarding Safari is outdated #32845

Closed
omar-rahman-dev opened this issue Mar 26, 2024 · 1 comment · Fixed by #35725
Closed

Navigator: cookieEnabled warning regarding Safari is outdated #32845

omar-rahman-dev opened this issue Mar 26, 2024 · 1 comment · Fixed by #35725
Labels
area: DOM/CSSOM Content:WebAPI Web API docs goal: up-to-date (Experimental label) Issues about outdated content. good first issue A good issue for newcomers to get started with.

Comments

@omar-rahman-dev
Copy link

MDN URL

https://developer.mozilla.org/en-US/docs/Web/API/Navigator/cookieEnabled

What specific section or headline is this issue about?

The first note

What information was incorrect, unhelpful, or incomplete?

Note: When the browser is configured to block third-party cookies, and navigator.cookieEnabled is invoked inside a third-party iframe, it returns true in Safari, Edge Spartan and IE (while trying to set a cookie in such scenario would fail). It returns false in Firefox and Chromium-based browsers.

What did you expect to see?

Safari technical preview 188 and latest webkit update has fixed navigator.cookieEnabled for Safari returning the correct value.

Do you have any supporting links, references, or citations?

https://developer.apple.com/documentation/safari-technology-preview-release-notes/stp-release-188
https://bugs.webkit.org/show_bug.cgi?id=267789

Do you have anything more you want to share?

No response
@omar-rahman-dev omar-rahman-dev added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Mar 26, 2024
@github-actions github-actions bot added the Content:WebAPI Web API docs label Mar 26, 2024
@Josh-Cena Josh-Cena changed the title Navigator: cookieEnabled property warning regarding Safari Navigator: cookieEnabled warning regarding Safari is outdated Jun 5, 2024
@Josh-Cena Josh-Cena added good first issue A good issue for newcomers to get started with. area: DOM/CSSOM goal: up-to-date (Experimental label) Issues about outdated content. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jun 5, 2024
@kaushik95
Copy link
Contributor

I would like to work on this issue.

kayhern pushed a commit to fidelity-contributions/mdn-content that referenced this issue Aug 28, 2024
@kaushik95
Navigator: cookieEnabled warning regarding Safari is outdated mdn#32845
8fb8d70 
Signed-off-by: Kowshik BV <kowshik.bv@fmr.com>
@kaushik95 kaushik95 mentioned this issue Sep 3, 2024
Merged
hamishwillee pushed a commit that referenced this issue Sep 12, 2024
@kaushik95
Navigator: cookieEnabled warning regarding Safari is outdated #32845 (#…
3fdf7ec 
…35725)

* Navigator: cookieEnabled warning regarding Safari is outdated #32845

Signed-off-by: Kowshik BV <kowshik.bv@fmr.com>

* Update index.md

Removed note section as it is not needed.

Signed-off-by: kaushik95 <kaushik.19995@gmail.com>

---------

Signed-off-by: Kowshik BV <kowshik.bv@fmr.com>
Signed-off-by: kaushik95 <kaushik.19995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: DOM/CSSOM Content:WebAPI Web API docs goal: up-to-date (Experimental label) Issues about outdated content. good first issue A good issue for newcomers to get started with.
Projects
None yet
Milestone
No milestone
3 participants
@kaushik95 @Josh-Cena @omar-rahman-dev