WPTT / theme-sniffer

Theme Sniffer plugin using sniffs.
MIT License
270 stars 3 forks source link

Add more specific indicators to theme name in dropdown list. #114

Closed timelsass closed 5 years ago

timelsass commented 5 years ago

Feature Request:

When using this - I often create a build zip of a final theme for distribution, deactivate my development versions and remove any hooks/filters I have and install the distribution version. This results in me having multiple themes in the dropdown list with the same name.

I think it would be handy to have the word (Active) next to the theme name, or perhaps the version number displayed next to it. Just something that can help identify multiple themes with the same theme name.

I realize the option is selected for the active theme, but when running multiple scans with different versions of the theme it's hard for me to keep track of. My current solution has been to just make a temp copy of my themes directory and test versions one by one to avoid the confusion.

dingo-d commented 5 years ago

I could add active and version number. The themes list business logic is done in the helpers trait so it shouldn't be a problem to do 🙂

Like the suggestion :+1:

dingo-d commented 5 years ago

screenshot 2019-03-02 at 19 01 48

Does this look good? @timelsass ?

timelsass commented 5 years ago

@dingo-d Yes - I like that

dingo-d commented 5 years ago

Ok I've pushed the changes to the PR, you can pull them. I'll close the issue 🙂