infrastructure/web-assets
1
4
Fork 3
Code Issues 3 Pull Requests 1 Projects Releases 41 Wiki Activity

BWA: Ordered Lists have no top margin #53407

New Issue
Closed
opened 2017-11-27 23:21:33 +01:00 by Aaron Carlisle · 7 comments
Aaron Carlisle commented 2017-11-27 23:21:33 +01:00
Contributor
Copy Link

The ordered lists <ol> does not have any top margin which is rendered differently than <ul>

Example HTML code:

<ol>
     <li>Ordered list item one</li>
     <li>Ordered list item two</li>
     <li>Ordered list item three</li>
</ol>

Example live page: https://www.blender.org/get-involved/documenters/
Compare "How to Get Started" with "Communication"

The ordered lists `<ol>` does not have any top margin which is rendered differently than `<ul>` Example HTML code: ``` <ol> <li>Ordered list item one</li> <li>Ordered list item two</li> <li>Ordered list item three</li> </ol> ``` Example live page: https://www.blender.org/get-involved/documenters/ Compare "How to Get Started" with "Communication"
Aaron Carlisle commented 2017-11-27 23:21:33 +01:00
Author
Contributor
Copy Link

Changed status to: 'Open'

Changed status to: 'Open'
Aaron Carlisle commented 2017-11-27 23:21:33 +01:00
Author
Contributor
Copy Link

Added subscribers: @Blendify, @pablovazquez, @fsiddi

Added subscribers: @Blendify, @pablovazquez, @fsiddi
Aaron Carlisle commented 2017-11-28 01:55:47 +01:00
Author
Contributor
Copy Link

The issue is here: https://developer.blender.org/diffusion/BWA/browse/master/styles/main.sass$457 if I can I will write a patch if not someone can take this up.

The issue is here: https://developer.blender.org/diffusion/BWA/browse/master/styles/main.sass$457 if I can I will write a patch if not someone can take this up.
Pablo Vazquez commented 2017-11-28 12:40:59 +01:00
Owner
Copy Link

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Pablo Vazquez self-assigned this 2017-11-28 12:40:59 +01:00
Pablo Vazquez commented 2017-11-28 12:40:59 +01:00
Owner
Copy Link

Fixed in production.

Just FYI, also fixed a typo in the title (Documen*tion) and renamed /documenters to /documentation with a redirect.

Fixed in production. Just FYI, also fixed a typo in the title (Documen*tion) and renamed /documenters to /documentation with a redirect.
Aaron Carlisle commented 2017-11-28 14:53:38 +01:00
Author
Contributor
Copy Link

Thank you!

Thank you!
Aaron Carlisle commented 2017-11-28 15:14:19 +01:00
Author
Contributor
Copy Link

There is still a 5px difference between p tags and the list tags.

p tags have 15px while list tags have 10px margins.

There is still a 5px difference between p tags and the list tags. p tags have 15px while list tags have 10px margins.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
legacy project
Infrastructure: Blender Web Assets

  
legacy project
Infrastructure: Websites

  
Priority
High

  
Priority
Low

  
Priority
Normal

  
Status
Archived

  
Status
Resolved

  
Type
Design

  
Type
Report

  
Type
To Do

No Label
legacy project
Infrastructure: Blender Web Assets
legacy project
Infrastructure: Websites
Priority
High
Priority
Low
Priority
Normal
Status
Archived
Status
Resolved
Type
Design
Type
Report
Type
To Do
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
Pablo Vazquez
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: infrastructure/web-assets#53407
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?