Panopoly is showing hidden themes on the theme selection step. Here's a patch to hide themes with hidden: true

CommentFileSizeAuthor
#2 panopoly_showing_hidden_themes.png113.04 KBshadcn
#1 panopoly_theme-hide-hidden-themes-2370539-1.patch1.26 KBshadcn

Comments

shadcn’s picture

shadcn commented
StatusFileSize
new panopoly_theme-hide-hidden-themes-2370539-1.patch1.26 KB
shadcn’s picture

shadcn commented
Issue summary: View changes
StatusFileSize
new panopoly_showing_hidden_themes.png113.04 KB
shadcn’s picture

shadcn commented
Component: Install » Theme
lsolesen’s picture

lsolesen commented
Status: Active » Needs review
klu’s picture

klu commented
Status: Needs review » Reviewed & tested by the community

Thanks @arshadcn for the patch. Confirming that the patch hides hidden themes from the Panopoly theme selection during the install process.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks @arshadcn and @klu! Patch looks good - committed. :-)

  • dsnopek committed bc1170b on 7.x-1.x 
    Update Panopoly Theme for #2370539 by arshadcn: Fixed Hide hidden themes...
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Reviewed & tested by the community » Fixed

Not sure why this status got punched back to RTBC. Moving back to Fixed, committed in #7

Status: Fixed » Closed (fixed)

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