The default view for the list of cards shown on the user profile is an "unformatted list", which I would assume on 80% of users won't be styled in a way that's usable "out of the box" with out some CSS love. I noticed you have no attached classes to this view, and there's no CSS in this module, so I can't imagine it supposed to pick up a theme from something like commerce kickstart, although I could be wrong.

The easiest way to make this page usable by most themes would be unfortunately to display it in a table. Any user who wants to change it back and add some CSS love to an unformatted list after that is welcome to, but at least out of the box with any theme which styles a table, it should look and be usable out of the box.

Attached is the view I created and exported.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dwkitchen’s picture

Status: Active » Needs review

I've had it as both as I have been doing the work on the module.

As we have gone for tables in Commerce Recurring, I will probably go for a table here to.

micheas’s picture

A warning, Last I checked. tables are generally an order of magnitude slower to be processed by javascript, so swapping out a list with a table could be the difference between an acceptable and unacceptable mobile experience.

j0rd’s picture

The current experience is unusable on any browser, as the information is displayed makes little sense to the end user.

Turning the view into a table by default solves this problem with minimal work. Should a site owner require a better mobile experience after that, then they could override the view. Additionally since most themes have tables styled in a reasonable fashion, this change gets this form auto styled on most sites.

Personally think point raised in #2 is not worth holding back this change.

deggertsen’s picture

Seems like a good idea to me. The current default view is pretty bad. I vote that this gets committed as it will most certainly be an improvement over what we have now.

mesch’s picture

+1 for this change.

deggertsen’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
FileSize
15.83 KB

I went ahead and created a patch out of this view change so that it can actually get committed. I made a few minor modifications that are mostly case changes so that the patch wouldn't make any unnecessary modifications.

Seeing as others have already looked at this view change I am marking as RTBC.

bojanz’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks.

  • Commit 06e7aaa on 7.x-2.x authored by deggertsen, committed by bojanz:
    Issue #2007646 by deggertsen, j0rd: Change the default user credit card...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.