Bug #19305

fact value page now has uncommon spacing

Added by Ohad Levy 9 months ago. Updated 3 months ago.

Status:Closed
Priority:Normal
Assigned To:Liza Gilman
Category:Facts
Target version:-
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/foreman/pull/4479, https://github.com/theforeman/foreman/pull/4711
Story points-
Velocity based estimate-
Release1.16.0Release relationshipAuto

Description

since the introduction of #19298, adding the breadcrumb css class adds un-needed spacing. see attached image.

fact.png (47.3 KB) Preview Ohad Levy, 04/18/2017 03:10 PM

1.png (12.9 KB) Preview Ohad Levy, 08/01/2017 11:32 AM

2.png (15.4 KB) Preview Ohad Levy, 08/01/2017 11:32 AM

3.png (8.05 KB) Preview Ohad Levy, 08/01/2017 11:32 AM

Fact 1 2 3

Related issues

Related to Foreman - Refactor #19298: Use chevron instead of colons for fact separators Closed 04/18/2017

Associated revisions

Revision d163e083
Added by Liza Gilman 6 months ago

Fixes #19305 - fixing spacing issue on facts page

Revision a8a0bd0e
Added by Liza Gilman 6 months ago

Refs #19305 - fixing spacing issues on facts page

History

#1 Updated by Ohad Levy 9 months ago

  • Related to Refactor #19298: Use chevron instead of colons for fact separators added

#2 Updated by Dominic Cleal 9 months ago

  • Category set to Facts

#3 Updated by Liza Gilman 9 months ago

  • Assigned To set to Liza Gilman

#4 Updated by The Foreman Bot 9 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4479 added

#5 Updated by Timo Goebel 6 months ago

  • Release set to 1.16.0

#6 Updated by Liza Gilman 6 months ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#7 Updated by Ohad Levy 6 months ago

  • Status changed from Closed to Feedback

IMHO this is not actually solved, see attached screenshots.

#8 Updated by Ohad Levy 6 months ago

#9 Updated by The Foreman Bot 6 months ago

  • Status changed from Feedback to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4711 added

#10 Updated by Daniel Lobato Garcia 3 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF