Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Up our table layout game #4621

Closed
4 tasks done
mazameli opened this issue Mar 27, 2017 · 18 comments
Closed
4 tasks done

Up our table layout game #4621

mazameli opened this issue Mar 27, 2017 · 18 comments
Assignees
Labels

Comments

@mazameli
Copy link
Contributor

mazameli commented Mar 27, 2017

I think the core team was all pretty keen on this article, and it made a few obvious suggestions that I think we should implement in our tables. Namely:

  • Numerical data should be right-aligned (!!!!)
  • Textual data should be left-aligned
  • Headers should be aligned with their data (i.e., headings for numerical fields should be right-aligned)
  • 3½. Don’t use center alignment.

I think we'll want to make exceptions for category fields that happen to be numerical — e.g., I don't think a phone number or ZIP code should be right-aligned.

It seems our typeface has pretty good, even spacing for numbers, so I think we're okay there but would be good to take another look to make sure things like decimal points and commas line up as they should.

@tlrobinson
Copy link
Contributor

tlrobinson commented Mar 27, 2017

For a second I thought you wanted us to port our HTML markup to use <table> for layout...

@mazameli
Copy link
Contributor Author

Well, it would improve cross-browser consistency… :trollface:

@mazameli
Copy link
Contributor Author

I also think we could experiment with getting rid of our vertical borders, slightly tightening the spacing in between rows, and slightly decreasing the font sizing. One wrinkle with this would be figuring out a way to indicate in the UI that you can resize the width of columns.

table-redesign

@salsakran
Copy link
Contributor

really like it visually. additionally, remember that table headings will have actions once #4496 is merged.

@tlrobinson
Copy link
Contributor

@mazameli how might pivoted tables look?

@mazameli
Copy link
Contributor Author

mazameli commented Apr 4, 2017

Yeah, as far as table heading actions, I think we can make the headings go blue on hover. (As an aside we'll have to figure out if sort-by-this-column is an action within the other actions menu, or if it's its own thing. I'm thinking the former right now.)

Pivot tables (i.e., 1 metric 2 dimension pivot tables) could look like this:

pivot table v1

@kdoh
Copy link
Member

kdoh commented Apr 4, 2017

Generally 👍 . Only thing that could get a bit odd in the case of the pivot example you posted is the alignment of the "x axis" pivoted field label and the right aligned labels for the different values. In the screenshot it's alignment with Apple is almost but not quite and it's wigging me out :P

@mazameli
Copy link
Contributor Author

mazameli commented Apr 4, 2017

Yeah, I waffled back and forth a bunch on where to align that label (Filling in the mock). Maybe we make it have the same alignment as the first heading in the x-axis, like so:

pivot table v1

@tlrobinson
Copy link
Contributor

tlrobinson commented Apr 4, 2017

Cool.

There was also the summary column/row in the new query builder design. I imagine they'd just look like the other rows/columns, with a slightly different bg color, and maybe fixed so they're always visible without scrolling?

@mazameli
Copy link
Contributor Author

mazameli commented Apr 4, 2017

Haven't thought that far TBH. In this issue I was trying to just propose CSS changes, and I see the subtotal/total cells as being a new feature. Good question though.

@tlrobinson
Copy link
Contributor

Resolved by #4977

@mazameli
Copy link
Contributor Author

I'll open a separate issue about stripping down the extraneous grid lines.

@EvaSchreyer
Copy link

Hi, I use metabase v0.25 and the alignment of colums is still not great:
image

As you can see I concat some of the numbers to add the "EUR" as there is currently no way to a) change the decimal dot to comma and b) to add a currency sign. Therefore the data is a string and is not aligned like the numbers in the other columns.

@EvaSchreyer
Copy link

Hi again, in case you still define the adjustments for the change it would be great if the user/the admin could decide if a column is right or left aligned.

The proposal:
Numerical data should be right-aligned (!!!!)
Textual data should be left-aligned

doesn´t really work when you still need concat syntax to add currency signs to your data - as then the number is turned into a string.

@cqcn1991
Copy link

If the alignment problem is solved? Seems still not.

@flamber
Copy link
Contributor

flamber commented Jul 15, 2019

@cqcn1991 Which version of Metabase? Can you provide a screenshot of your problem or more details?

@Dcushing57
Copy link

Hi,

It does not appear the the pivot table headers do not use right alignment when value's data type is Number. I am also using 3 category data types, 2 in the column, 1 in the row. Neither header is right aligned.

I received a complaint that the pivot table is hard to read as it almost looks like things are misaligned due to the left alignment on both headers and right alignment on number.

I am using a pivot table with the following details. I'm sorry, I cannot provide screenshot due to PHI. I will try to describe details as close as I can.

Summarization:

Sum(Number data type) by Category 1, Category 2, Category 3

Rows:
Category 1

Columns:
Category 2
Category 3

Values:
Sum(Number)

@flamber
Copy link
Contributor

flamber commented Jun 30, 2021

@Dcushing57 This issue is closed and several years old. You are seeing this issue: #16752

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants