Introduce `Support:` syntax tagging Browser Hacks in source code

Created on 14 October 2016, about 8 years ago
Updated 23 January 2023, almost 2 years ago

Problem/Motivation

Improve code comments to save our efforts on re-evaluating the Browser supports. It helps to analysis Browser Hacks usages in CORE & Cleaning up in future releases.

This is how jQuery does also:
https://unpkg.com/jquery@3.1.1
searching `// Support:`

For example,

the following patch I think OR condition is unnecessary but we don't have 100% confidence to remove it:
https://www.drupal.org/node/2751643#comment-11682503

Another example:
@import hack in CORE is also outdated I think:
https://www.drupal.org/node/2765525#comment-11404573

Proposed resolution

- (Inform our CORE developer with right channel)

Remaining tasks

- (Discuss: Not rush for current source and only used in new hacks?)

🌱 Plan
Status

Active

Component

Coding Standards

Created by

🇭🇰Hong Kong droplet

Live updates comments and jobs are added and updated live.
  • CSS

    It involves the content or handling of Cascading Style Sheets.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024