katiefenn / parker

Stylesheet analysis tool.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Feature: Add z-index values to report

davegreenwp opened this issue · comments

I think this would be useful to see if you've ended up with ridiculous z-indexes! 👍

Not sure how to list these tho. Show highest? Show total?
I have a little postcss-script here that lists all z-index properties, which can be a good thing to check once in a while. But maybe it’s a but much for this plugin?

Or show all unique values, like here? (personal very early side project): https://projectwallace.herokuapp.com/users/1/projects/8/metrics/unique_zindexes

@vincentorback @bartveneman Personally I'd like to see the highest value and the total number of unique values. I don't think anything else is really all that useful. If you see that you've got high figure of 9999 or you have 150 (for example) different values then you know that's silly and you need to intervene.

That sounds reasonable, although I've seen some projects using groupings like 100-103 for site headers, 200-204 for some component and 900-910 for modals. As always, I guess: it depends

Yeah, it’s a good thing to see all of your z-index's to see that they stack with a reasonable amount. But I’m not sure it’s parkers job to show this. So maybe showing highest and total is good enough!

Personally I'd like to see the highest value and the total number of unique values.

But I’m not sure it’s parkers job to show this. So maybe showing highest and total is good enough!

I think you're both right. Parker usually shows averages/means, so deviating from that would be indeed a little strange.