Skip to content

Update Symfony browserlists reccomendation to browserlists standard. #14076

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

Merged
merged 1 commit into from
Aug 31, 2020

Conversation

Taminoful
Copy link
Contributor

@Taminoful Taminoful commented Aug 14, 2020

Use the browserlists default browsers as recommended by "browserlists"

The defaults option of browserlists includes the already mentioned settings on this page and is the recommended value for most users, listing all the including values as a recommendation from Symfony is not helpful. I included the equivalent of defaults below the new recommendation for explanation.

See: Full List of Browserslist

@Taminoful Taminoful changed the title Update postcss.rst Update Symfony browserlists reccomendation to browserlists standard. Aug 14, 2020
@javiereguiluz javiereguiluz changed the base branch from 5.1 to 3.4 August 31, 2020 08:08
@javiereguiluz
Copy link
Member

This change makes a lot of sense! Thanks Robin and congrats on your first Symfony Docs contribution!

@javiereguiluz javiereguiluz merged commit a60a2cb into symfony:3.4 Aug 31, 2020
@xabbuh xabbuh added this to the 3.4 milestone Aug 31, 2020
xabbuh added a commit that referenced this pull request Aug 31, 2020
xabbuh added a commit that referenced this pull request Sep 7, 2020
* 3.4:
  typo
  [#14076] fix typos
xabbuh added a commit that referenced this pull request Sep 7, 2020
* 4.4:
  typo
  do not document adding attachments through URLs
  Migrate Yaml config for DI calls to new syntax
  [#14076] fix typos
xabbuh added a commit that referenced this pull request Sep 7, 2020
* 5.1:
  typo
  do not document adding attachments through URLs
  Migrate Yaml config for DI calls to new syntax
  [#14076] fix typos
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants