Make WordPress Core

Opened 8 days ago

Last modified 6 days ago

#61963 new defect (bug)

Twenty Twenty-One: defining standard `appearance` property in the CSS files

Reported by: viralsampat's profile viralsampat Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch reporter-feedback close 2nd-opinion
Focuses: css Cc:

Description

Hello Team,

I have reviewed twenty twenty-one theme and found CSS compatibility changes into the CSS file.

So, I think that it should be resolve it for consistency and standard.

Thanks,

Attachments (1)

61963.patch (5.0 KB) - added by viralsampat 8 days ago.
I have checked above mentioned issue and founds few files. Here, I have added its patch.

Download all attachments as: .zip

Change History (3)

@viralsampat
8 days ago

I have checked above mentioned issue and founds few files. Here, I have added its patch.

#1 @dd32
8 days ago

  • Keywords reporter-feedback added; dev-feedback has-testing-info removed

Hello @viralsampat,

The CSS files you've edited in the attached patch are generated from the files within the assets/sass folder.

It appears that the rules you've altered all come from the external library, normalize.

/*! normalize.css v8.0.1 | MIT License | github.com/necolas/normalize.css */

The CSS rules from normalize are intentionally not cross-browser, as they exist purely to "normalize" the display in various browsers.

Is there a specific problem which you're attempting to resolve with this ticket? or is it purely formatting and consistency? If so, I believe this ticket is actually invalid as normalize is intentional.

#2 @sabernhardt
6 days ago

  • Keywords close 2nd-opinion added
  • Summary changed from Twenty twenty-one theme: I have defined standard property into the CSS file. to Twenty Twenty-One: defining standard `appearance` property in the CSS files

The patch involves more than normalize.css, but I would not recommend the proposed changes.

Last edited 6 days ago by sabernhardt (previous) (diff)
Note: See TracTickets for help on using tickets.