Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#43024 closed defect (bug) (invalid)

Twenty Fifteen: ul li > blockquote has styling issues

Reported by: jorbin Owned by: junaidkbr
Priority: normal Milestone:
Component: Bundled Theme Version:
Severity: normal Keywords: needs-design good-first-bug needs-testing has-patch
Cc: Focuses:

Description

See ​https://twitter.com/mor10/status/949044243951099904

The dot and the blockquote border seem to overlap. May also affect ol li > blockquote

Attachments (1)

43024.patch​ (1.3 KB ) - added by Junaidkbr 9 years ago.
Negative margin-left removed for ul/ol > li > blockquote for both frontend and editor

Download all attachments as: .zip

Change History (10)

@Junaidkbr
9 years ago

Negative margin-left removed for ul/ol > li > blockquote for both frontend and editor

#1 @Junaidkbr
9 years ago

  • Keywords needs-testing has-patch added

#2 @mcsmithers
9 years ago

I'd like to help out. It will be the first time I help our Wordpress, so any suggestions you have are welcome. I see we still need testing. Are we thinking unit tests?

This ticket was mentioned in ​Slack in #core-themes by junaidkbr. ​View the logs.


9 years ago

#4 @DrewAPicture
9 years ago

  • Owner set to junaidkbr
  • Status new → assigned

Assigning to mark the good-first-bug as "claimed".

#5 @ketanumretiya030
9 years ago

  • Component Bundled Theme → Formatting
  • Keywords close added
  • Resolution → invalid
  • Status assigned → closed

I have test and its working fine.

#6 follow-up: @SergeyBiryukov
9 years ago

  • Component Formatting → Bundled Theme
  • Keywords close removed
  • Resolution invalid
  • Status closed → reopened
  • Summary ul li > blockquote has styling issues in twentyfifteen → Twenty Fifteen: ul li > blockquote has styling issues

Could not reproduce the overlapping in Opera, though the overall appearance does seem a bit off.

@mor10 Which browser was the overlapping in?

#7 in reply to: ↑ 6 @mor10
9 years ago

Replying to SergeyBiryukov:

@mor10 Which browser was the overlapping in?

Chrome

#8 @ketanumretiya030
9 years ago

  • Resolution → invalid
  • Status reopened → closed

Hi,

i have check in all browser never found issue.

#9 @netweb
8 years ago

  • Milestone 5.0
Note: See TracTickets for help on using tickets.