Tag improvements

Configuration
21

@Terdious
Thanks, that’s exactly what I thought :slight_smile:

Many sites that use tags work cumulatively.
Ex: I want all products from the

22

It all depends on what you want to use it for.
You’re assuming that you want to compare two products.
But in Gladys you can’t compare two scenes.
I just want to list the scenes that share the

23

The thing is we’re talking about two fairly different concepts.

I personally see tags as a filter and others see them as a way to sort. A filter works like a funnel: the more you add, the more it narrows what comes out.
So it seems logical to me that they work like an AND.

In fact, nothing is impossible even with this behavior:

@Terdious for HP / HC, you just need to create a « higher-level Â» tag that could be called Plages HP-HC (I’m saying this off the top of my head, you’d want something more fitting and meaningful to you).

So you tag your scenes:

For peak hours: Plages HP-HC + HP
For off-peak hours: Plages HP-HC + HC

Which is ultimately what we all more or less did by tinkering with the names of our scenes to find them.

And then you can do OR by checking your HP or HC tag and AND by checking Plages HP-HC +

1 Like
24

For me it’s the behavior that seems the most obvious


Even though I understand well what you’re saying @Hizo
The arguments of @guim31 and @spenceur also make sense.
Like @spenceur, I also came to understand another perspective in the process.

Consensus will be difficult; in any case, some people will have to add more tags to find their way.

If we had to vote, I’d go for AND. (some kind of Doodle poll for that?)

25
  • When selecting multiple tags in the Scenes view, only the scenes linked to ALL the tags are returned (AND).
  • When selecting multiple tags in the Scenes view, all scenes linked to AT LEAST one tag are returned (OR).
0 votant

Cast your votes :wink:

7 Likes
26

Great, magical, effective!

1 Like
27

I left you a review: Fix return tag scene by callemand · Pull Request #1946 · GladysAssistant/Gladys · GitHub

1 Like
28

@pierre-gilles : Thanks. I just had the feedback corrected

29

I think we can go with it in the end.
Unless there’s a way to satisfy everyone, but I don’t think that’s the philosophy of making everything customizable

1 Like
30

Thanks! I’ve re-reviewed it: Fix return tag scene by callemand · Pull Request #1946 · GladysAssistant/Gladys · GitHub

31

Thanks @Lokkye for your latest feedback, it’s tested and approved on my end :rocket:

It’s merged and will ship in the next Gladys release!

2 Likes
32

The fixes are available in the latest version of Gladys!