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

Show table borders on action columns #2165

Merged
merged 1 commit into from
Aug 18, 2017

Conversation

jhawthorn
Copy link
Contributor

Previously, the action icons were shown clearly outside the table row they belonged to. I suspect this is just because it looked bad.

With the new table design, I believe it looks better to group the actions under the same row separator as the rest of the row, and it makes it more obvious what it is operating on.

This also right-aligns the action cells.

Before

Note extra whitespace at right

After

Previously, the action icons were shown clearly outside the table row
they belonged to. I suspect this is just because it looked bad.

With the new table design, I believe it looks better to group the
actions under the same row separator as the rest of the row, and it
makes it more obvious what it is operating on.

This also right-aligns the action cells.
@jhawthorn jhawthorn added changelog:solidus_backend Changes to the solidus_backend gem UI labels Aug 17, 2017
@tvdeyen
Copy link
Member

tvdeyen commented Aug 17, 2017

Some people may find it weird that the actions are right aligned, especially on tables with unequal actions item counts

payments - r987654321 - orders 2017-08-17 22-53-52

shipments - r987654321 - orders 2017-08-17 22-54-13

I like it 👍

Copy link
Contributor

@cbrunsdon cbrunsdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks better to me, for whatever thats worth!

@Mandily
Copy link
Contributor

Mandily commented Aug 17, 2017

When thinking about the alignment of the action buttons - the thing that I would most want to align to is the same action above or below the current row. So a particular action could always be found by scanning straight down. It's one more little piece (location) that helps a user identify what the action is. Kind of like how a colour blind person knows that the top light on a traffic light is stop even though they don't see red.

For example: all of the X's would line up straight down the side.
29433353-2071b138-839f-11e7-9ccd-3e6d90f5720c

Thoughts?

@jhawthorn
Copy link
Contributor Author

jhawthorn commented Aug 17, 2017

Aligning the identical actions looks good (for the cases it makes sense), but I suspect we'd have to do it with some sort of placeholder element. This PR won't help or hurt our ability to do that in the future.

Copy link
Contributor

@mamhoff mamhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I like it better too.

@tvdeyen tvdeyen merged commit 4c37c82 into solidusio:master Aug 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_backend Changes to the solidus_backend gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants