search

OpenTechStrategies / arches-app-exp

An Arches Application, for experimenting with Arches Applications.
GNU Affero General Public License v3.0
1 stars 0 forks source link

Exclude current artwork from "More artwork" section #89

Open reefdog opened 2 days ago

reefdog commented 2 days ago

On a resource detail page, we have a "More artwork by [artist]" block. This includes all artwork by the artist, inclusive of the current artwork you're viewing.

Preferably, this should only show other artwork by the same artist (and hide entirely if there is none).

But also acceptable if that's too difficult is just to rename this section to "All artwork by [artist]".

hminsky2002 commented 2 days ago

Excellent write up of the issue, I should've been more explicit but this was actually closed by #86 ! More artworks will only display if there are more than one now commit

reefdog commented 2 days ago

I did later realize you had checked this off your list — for some reason I thought I wasn't seeing this locally, but I am now, so, yay!

That said, I did notice that while it says "More artwork by…" it actually shows all artwork by. As mentioned in the description above, ideally it would exclude the current artwork — failing that, it should be titled "All artwork by…". But I'll bet us geniuses can figure out how to exclude the current one.