Fix wording and formatting inconsistencies in cookies.set documentation by Iyeroo · Pull Request #43107 · mdn/content

Iyeroo · 2026-02-12T17:57:21Z

Description

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set#expirationdate
The above link redirects to expirationdate of cookie the statement was grammatically incorrect it started with can include milliseconds' there was no pronoun prior to it making it sound subordinate.

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set#httponly

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set#secure

The above links followed true in <code> not following same for false making it inconsistent so I included false as well in <code> to maintain the consistency.

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set#name

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set#value

The meaning to the line in above links (if omitted, this is empty by default) was unclear to make it sound the clear I replaced it with the below line:

If omitted, the value defaults to an empty string ("").

Motivation

I like to read documentation that carry clarity and easy to understand so this made me make these changes which get rid of ambiguity readers faced for while.

Additional details

Related issues and pull requests

Fixes #42988

Rob--W · 2026-02-12T20:24:22Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

  - : A `string` representing the domain the cookie belongs to (e.g., "www.google.com" or "example.com").
 - `expirationDate` {{optional_inline}}
-  - : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch. Includes milliseconds in the fractional part. Not provided for session cookies.
+  - : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.


The last

Suggested change

- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.

- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch. It can include milliseconds in the fractional part. Not provided for session cookies.

This type describes the return value of various cookie APIs. The last fragment was copied from the cookies.set description, where "becomes" makes sense because the method creates or updates a cookie. When a cookie is read, "becomes" feels odd, because the cookie does not become anything, its state is returned as is.

thankyou for the feedback I will revise it

@Iyeroo, please also refer to the feedback you received from me on Object #43110.

Hello Maintainers,
I referred to your feedback and understood that set method is responsible to make a cookie session cookie not cookies.cookie() object and have made changes with respect to the same reference.Waiting for review

…es.cookie() and cookies.set()

rebloor · 2026-02-22T15:43:39Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

  - : A `string` representing the domain the cookie belongs to (e.g., "www.google.com" or "example.com").
 - `expirationDate` {{optional_inline}}
-  - : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch. Includes milliseconds in the fractional part. Not provided for session cookies.
+  - : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch.Can include milliseconds in the fractional part.Not provided for session cookies.


Suggested change

- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch.Can include milliseconds in the fractional part.Not provided for session cookies.

- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch. Includes milliseconds in the fractional part. Not provided for session cookies.

The fractional part always includes the milliseconds value from the expiration date.

I understood the fact and revised it waiting for review.

rebloor · 2026-02-22T15:45:26Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

+  - : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or false otherwise.
 - `name`
-  - : A `string` representing the name of the cookie.
+  - : A `string` representing the name of the cookie.If omitted, this is empty by default.


Suggested change

- : A `string` representing the name of the cookie.If omitted, this is empty by default.

- : A `string` representing the name of the cookie.

As this is the return value of the cookie name can't be "omitted".

I understood that whatever is returned as an object cannot be omitted and made the changes.

rebloor · 2026-02-22T15:46:57Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

  - : A `boolean`, `true` if the cookie is a host-only cookie (i.e., the request's host must exactly match the domain of the cookie), or `false` otherwise.
 - `httpOnly`
-  - : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or `false` otherwise.
+  - : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or false otherwise.


Suggested change

- : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or false otherwise.

- : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or `false` otherwise.

As true and false are returned values they should both be in code format.

I have included both in code format.Waiting for review.

rebloor · 2026-02-22T15:47:35Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

  - : A `string` representing the path of the cookie.
 - `secure`
-  - : A `boolean`, `true` if the cookie is marked as secure (i.e., its scope is limited to secure channels, typically HTTPS), or `false` otherwise.
+  - : A `boolean`, `true` if the cookie is marked as secure (i.e., its scope is limited to secure channels, typically HTTPS), or false otherwise.


Suggested change

- : A `boolean`, `true` if the cookie is marked as secure (i.e., its scope is limited to secure channels, typically HTTPS), or false otherwise.

- : A `boolean`, `true` if the cookie is marked as secure (i.e., its scope is limited to secure channels, typically HTTPS), or `false` otherwise.

I made suggested changes.

rebloor · 2026-02-22T15:47:57Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

  - : A `string` representing the ID of the cookie store containing this cookie, as provided by {{WebExtAPIRef("cookies.getAllCookieStores()")}}.
 - `value`
-  - : A `string` representing the value of the cookie.
+  - : A `string` representing the value of the cookie. If omitted, this is empty by default.


Suggested change

- : A `string` representing the value of the cookie. If omitted, this is empty by default.

- : A `string` representing the value of the cookie.

I revised it.

rebloor · 2026-02-22T15:48:29Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/set/index.md

      - : A `string` representing the domain of the cookie. If omitted, the cookie becomes a host-only cookie.
    - `expirationDate` {{optional_inline}}
-      - : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch. Can include milliseconds in the fractional part. If omitted, the cookie becomes a session cookie.
+      - : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.


Suggested change

- : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.

- : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch. It can include milliseconds in the fractional part. If omitted, the cookie becomes a session cookie.

It has been made.

rebloor · 2026-02-22T15:48:55Z

files/en-us/mozilla/add-ons/webextensions/api/cookies/set/index.md

+      - : A `boolean` that specifies whether the cookie is marked as HttpOnly (`true`), or not (`false`). If omitted, it defaults to `false`.
    - `name` {{optional_inline}}
-      - : A `string` representing the name of the cookie. If omitted, this is empty by default.
+      - : A `string` representing the name of the cookie.If omitted, the name defaults to an empty string (`""`).


Suggested change

- : A `string` representing the name of the cookie.If omitted, the name defaults to an empty string (`""`).

- : A `string` representing the name of the cookie. If omitted, the name defaults to an empty string (`""`).

Hello maintainers,
I have made all changes suggested.waiting for review.

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md

The necessary changes have now been made

github-actions · 2026-02-24T22:58:30Z

Preview URLs (1 page)

/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/set

rebloor

Thanks for your contribution @Iyeroo

Fix wording and formatting inconsistencies in cookies.set documentation

6c3ed25

Iyeroo requested a review from a team as a code owner February 12, 2026 17:57

Iyeroo requested review from Rob--W and removed request for a team February 12, 2026 17:57

github-actions bot added Content:WebExt WebExtensions docs size/s [PR only] 6-50 LoC changed labels Feb 12, 2026

Rob--W reviewed Feb 12, 2026

View reviewed changes

Iyeroo added 3 commits February 13, 2026 19:47

cookies.cookie were brought back to original state

33d5143

the changes made in cookies.set()

61ae755

feedback on object mdn#43110 was considered and changes made to cooki…

ed6bd88

…es.cookie() and cookies.set()

rebloor requested changes Feb 22, 2026

View reviewed changes

Requested changes committed

3a2da01

rebloor previously requested changes Feb 22, 2026

View reviewed changes

files/en-us/mozilla/add-ons/webextensions/api/cookies/cookie/index.md Outdated Show resolved Hide resolved

rebloor added 2 commits February 25, 2026 11:55

Apply suggestion from @rebloor

c732543

Merge branch 'main' into anisha

7acf3a2

rebloor approved these changes Feb 24, 2026

View reviewed changes

rebloor merged commit 32823bb into mdn:main Feb 24, 2026
8 checks passed

rebloor mentioned this pull request Feb 24, 2026

cookies.set(): minor improvements #42988

Closed

	- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.
	- : A `number` representing the expiration date of the cookie as the seconds after the UNIX epoch. It can include milliseconds in the fractional part. Not provided for session cookies.

	- : A `string` representing the name of the cookie.If omitted, this is empty by default.
	- : A `string` representing the name of the cookie.

	- : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or false otherwise.
	- : A `boolean`, `true` if the cookie is marked as HttpOnly (i.e., the cookie is inaccessible to client-side scripts), or `false` otherwise.

	- : A `string` representing the value of the cookie. If omitted, this is empty by default.
	- : A `string` representing the value of the cookie.

	- : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch.It can include milliseconds in the fractional part.If omitted, the cookie becomes a session cookie.
	- : A `number` that represents the expiration date of the cookie as the seconds after the UNIX epoch. It can include milliseconds in the fractional part. If omitted, the cookie becomes a session cookie.

Conversation

Iyeroo commented Feb 12, 2026 • edited by rebloor Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Motivation

Additional details

Related issues and pull requests

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Feb 24, 2026

Uh oh!

rebloor left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Iyeroo commented Feb 12, 2026 •

edited by rebloor

Loading