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

Consider ways to reduce clutter in "All Popups" table #878

Closed
fpcorso opened this issue Sep 15, 2020 · 3 comments
Closed

Consider ways to reduce clutter in "All Popups" table #878

fpcorso opened this issue Sep 15, 2020 · 3 comments

Comments

@fpcorso
Copy link
Contributor

fpcorso commented Sep 15, 2020

With the addition of the enabled toggle, the "All Popups" page is getting a little cluttered when one has tags/categories enabled too, which we know almost 6% of our customers do. This is what it looks like on my laptop screen which is definitely a bit cluttered:
laptop-example

Even on my larger monitor with the sidebar closed:
monitor-example

We also wanted to add in the conversion counter (see #775) columns too which would make this even more cluttered.

Since we don't recommend people ever use popmake-{name}, do you think we could remove that from the CSS classes? Other than that, the only thing else we could consider removing is the whole CSS classes column or the title column. If we just remove the second CSS class, it wouldn't be too bad on the larger monitor but would be still a bit tight after we add conversion counters.

Alternatively (or additionally), we may be able to get rid of the Title or published date.

The other main question is: Do people use "Screen Options" to adjust these? If so, it's not something we'd have to worry about too much. My hunch is that most people see the clutter and do not think of the "Screen Options".

@fpcorso fpcorso added this to the v1.13 milestone Sep 15, 2020
@danieliser
Copy link
Member

  1. We can remove the popmake-name classes from the list here, they still need to function for back-compat but I'm good with the other proposed change.

  2. Agree, we can probably ditch or at least mark Title and published date as not visible by default under screen options.

  3. Take it a step further, extend the Admin Pointers to show they can change what data is visible and available to them via Screen Options.

@fpcorso
Copy link
Contributor Author

fpcorso commented Oct 2, 2020

@danieliser Removed the extra css class from the table, hidden title and date by default, and added the extra pointer.

image

fpcorso added a commit that referenced this issue Oct 2, 2020
@fpcorso fpcorso closed this as completed Oct 2, 2020
@danieliser
Copy link
Member

Nicely done.

@fpcorso fpcorso mentioned this issue Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants