Skip to content

Display all Collection Tags on a Dataset - #231

Merged
phargogh merged 1 commit into
masterfrom
feature/multi-collection-display
May 1, 2026
Merged

Display all Collection Tags on a Dataset#231
phargogh merged 1 commit into
masterfrom
feature/multi-collection-display

Conversation

@megannissel

Copy link
Copy Markdown
Contributor

Previously, the templates (both the dataset list page and the dataset details page) were just displaying (and linking) the first collection tag on a dataset in the tile / sidebar. But I noticed @empavia had tagged the Historic Koppen-Geiger Climate Zone datasets (ex) with both KOPPEN-GEIGER CLIMATE ZONES and KOPPEN-GEIGER CLIMATE ZONES HISTORIC. There's no reason this shouldn't be supported, and it's a feature I had wanted to add at some point anyway, so I figured I'd just go ahead and do it.

Now, all collection tags should display in the sidebar, with links to either the associated Collection Package (if relevant) or a filtered search (if no associated Collection Package). See attached screenshots. (And ignore the fact that the tags here don't make sense; this was just for local testing!)

Screenshot 2026-05-01 at 10 46 15 AM Screenshot 2026-05-01 at 10 46 24 AM

@megannissel
megannissel requested review from empavia and phargogh May 1, 2026 14:49
@empavia

empavia commented May 1, 2026

Copy link
Copy Markdown
Contributor

Thanks, Megan! Adding two collection tags was something I was playing around with yesterday while thinking about data organization. I do think it is helpful, and I am happy to see you agree and are already on top of the UI changes! This looks great to me.

@phargogh phargogh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @megannissel !

@phargogh
phargogh merged commit a834b72 into master May 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants