[2.4] Incorrect table spacing causes wrapping in wrong places

Bug #1751908 reported by Andres Rodriguez
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
High
Karl Williams

Bug Description

Incorrect table spacing causes wrapping in wrong places. This is across the board, just putting one example.

Related branches

Revision history for this message
Andres Rodriguez (andreserl) wrote :
tags: added: vanilla-transition
Revision history for this message
Andres Rodriguez (andreserl) wrote :

orig to compare

Changed in maas:
importance: Undecided → Medium
status: New → Triaged
milestone: none → 2.4.0alpha2
summary: - [2.4] Incorrect table spacing causes wrapping in wrong plcaes
+ [2.4] Incorrect table spacing causes wrapping in wrong places
Revision history for this message
Andres Rodriguez (andreserl) wrote :

ANother 2.4 bad wrapping example.

Changed in maas:
importance: Medium → High
Revision history for this message
Anthony Dillon (ya-bo-ng) wrote :

The reason for the change in tables is due to Vanilla using the grid to layout the table. Before each column had a bespoke width but to achieve horizontal rhythm thought out the content we use a 12 column grid. This may cause some wrapping in tables but case by case we can either redistribute the columns to reduce wrapping or move data within the revealed row.

Revision history for this message
Lyubomir Popov (lyubomir-popov) wrote :

I agree with Ant, having a table aligned to the grid is better,
unless it causes severe compromises with the content.

In this particular example I'd explore shortening content as much as possible - Luke mentioned a js library that helps with that - I think it was this one - https://momentjs.com

Could we get a full list of examples and address case by case?

Revision history for this message
Andres Rodriguez (andreserl) wrote : Re: [Bug 1751908] Re: [2.4] Incorrect table spacing causes wrapping in wrong places

I don’t think we need a new js library to fix tables. We shouldn’t be
finding a work around in JS to fix something that has to do with formatting.

I’m considering this a regression provided that this is affecting tables
across the board and is not organizing the data as it should be (e.g for
fields that don’t r quite a lot of space that don’t need it, or for fields
who require more space and don’t have it because others are taking space
when they shouldn’t.)

On Wed, Feb 28, 2018 at 8:41 AM Lyubomir Popov <email address hidden>
wrote:

> I agree with Ant, having a table aligned to the grid is better,
> unless it causes severe compromises with the content.
>
> In this particular example I'd explore shortening content as much as
> possible - Luke mentioned a js library that helps with that - I think it
> was this one - https://momentjs.com
>
> Could we get a full list of examples and address case by case?
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1751908
>
> Title:
> [2.4] Incorrect table spacing causes wrapping in wrong places
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/maas/+bug/1751908/+subscriptions
>
> Launchpad-Notification-Type: bug
> Launchpad-Bug: product=maas; milestone=2.4.0alpha2; status=Triaged;
> importance=High; assignee=None;
> Launchpad-Bug-Tags: vanilla-transition
> Launchpad-Bug-Information-Type: Public
> Launchpad-Bug-Private: no
> Launchpad-Bug-Security-Vulnerability: no
> Launchpad-Bug-Commenters: andreserl lyubomir.popov ya-bo-ng
> Launchpad-Bug-Reporter: Andres Rodriguez (andreserl)
> Launchpad-Bug-Modifier: Lyubomir Popov (lyubomir.popov)
> Launchpad-Message-Rationale: Subscriber
> Launchpad-Message-For: andreserl
>
--
Andres Rodriguez (RoAkSoAx)
Ubuntu Server Developer
MSc. Telecom & Networking
Systems Engineer

Changed in maas:
milestone: 2.4.0alpha2 → 2.4.0beta1
Changed in maas:
assignee: nobody → Karl Williams (deadlight)
status: Triaged → In Progress
Changed in maas:
milestone: 2.4.0beta1 → 2.4.0beta2
Changed in maas:
status: In Progress → Fix Committed
Changed in maas:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.