Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#46821 closed enhancement (fixed)

Twenty Nineteen: Fix inconsistent tab/space indentation in some scss

Reported by: iamdmitrymayorov Owned by: SergeyBiryukov
Priority: normal Milestone: 5.2
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch has-screenshots
Cc: Focuses:

Description

There are two lines in _blocks.scss in Twenty Nineteen theme that have indentation issues. The first one has redundant tabs and the second one has mixed tab/space indentation.

It looks like the second line was copied from another document with different indentation style.

Not a huge deal, but probably still worth fixing.

Attachments (2)

46821.diff​ (603 bytes ) - added by iamdmitrymayorov 8 years ago.
46821.patch​ (11.7 KB ) - added by mukesh27 8 years ago.
Added patch that fix inconsistent tab/space indentation in remaining scss files.

Download all attachments as: .zip

Change History (6)

#1 @iamdmitrymayorov
8 years ago

It might not be very clear from the patch preview, so here's a screenshot as well: ​https://monosnap.com/file/RSq3zrNh42uFQJzxm0RCDgQFWynQqV

#2 @SergeyBiryukov
8 years ago

  • Milestone Awaiting Review → 5.2
  • Owner set to SergeyBiryukov
  • Status new → reviewing

#3 @mukesh27
8 years ago

  • Keywords has-patch has-screenshots added
  • Summary Twenty Nineteen: Fix inconsistent tab/space indentation in _blocks.scss → Twenty Nineteen: Fix inconsistent tab/space indentation in some scss

Updated title as some files are also contain inconsistent tab/space indentation so that also needs to fix.

@mukesh27
8 years ago

Added patch that fix inconsistent tab/space indentation in remaining scss files.

#4 @SergeyBiryukov
8 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 45132:

Twenty Nineteen: Fix inconsistent tab/space indentation in style files.

Props iamdmitrymayorov, mukesh27.
Fixes #46821.

Note: See TracTickets for help on using tickets.