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

Table <td> alignment issue noticed on Payment methods page #188

Closed
jacine opened this issue Jul 8, 2019 · 5 comments
Closed

Table <td> alignment issue noticed on Payment methods page #188

jacine opened this issue Jul 8, 2019 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@jacine
Copy link
Contributor

jacine commented Jul 8, 2019

Describe the bug
<td> alignment is inconsistent, I think partially because of .btn-link / operations. I just ran into this somewhere else that I cannot recall ATM, but we should fix this globally if we can.

To Reproduce
Steps to reproduce the behavior:

  1. Log in and go to "Payment methods" page.
  2. Add one or more payment methods.
  3. Notice the the <td> elements are not aligned consistently.

Expected behavior
Should probably all align top vertically?

Screenshots
payment-methods-table-styling-off

@jacine jacine added the bug Something isn't working label Jul 8, 2019
@jacine jacine added this to the 8.x-1.3 milestone Jul 8, 2019
@jacine jacine added this to To do in Drupal-based Development Board via automation Jul 8, 2019
@shadcn
Copy link
Contributor

shadcn commented Jul 9, 2019

Looks like commerce is replacing dropbuttons with normal links. See \Drupal\commerce_payment\PaymentMethodListBuilder::buildOperations.

We could go back to dropbutton? This looks cleaner but might still need some alignment fixes.

Payment_methods___Apigee_Inc

@jacine
Copy link
Contributor Author

jacine commented Jul 9, 2019

Hmm... 🤔 What do you think the odds of them changing this in the future are? Looks like this is a standard, rigid entity list, that can barely be manipulated at all, so it may be safe...

I guess another option is NOT to use .btn-link and use/create a button style that looks like this, e.g. btn-tertiary?

shadcn added a commit to shadcn/apigee-devportal-kickstart-drupal that referenced this issue Jul 10, 2019
@shadcn
Copy link
Contributor

shadcn commented Jul 10, 2019

I guess another option is NOT to use .btn-link and use/create a button style that looks like this, e.g. btn-tertiary?

Let's see if we see this again then we can add a new .btn-tertiary.

For now, I removed the padding with the buttons. Let me know what you think.

@shadcn shadcn self-assigned this Jul 10, 2019
@shadcn shadcn moved this from To do to Review in progress in Drupal-based Development Board Jul 10, 2019
jacine added a commit that referenced this issue Jul 10, 2019
[#188] Update style for nav link inside table
@jacine
Copy link
Contributor Author

jacine commented Jul 10, 2019

Fixed by #199.

@jacine jacine closed this as completed Jul 10, 2019
Drupal-based Development Board automation moved this from Review in progress to Done Jul 10, 2019
@jacine
Copy link
Contributor Author

jacine commented Jul 10, 2019

I guess another option is NOT to use .btn-link and use/create a button style that looks like this, e.g. btn-tertiary?

Let's see if we see this again then we can add a new .btn-tertiary.

For now, I removed the padding with the buttons. Let me know what you think.

Yes, this works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

2 participants