Sieve filter dialog missing operation when "other header" is selected #23

New Issue

Open

opened 2026-08-25 08:36:49 +00:00 by mbunkus · 2 comments

mbunkus commented

2026-08-25 08:36:49 +00:00

When editing Sieve filters I can select "other header" & enter my own header name to filter on. However, that replaces the comparator/operation (contains/matches/…).

git at b194bc0a33cc2e02ef2ac53ebae03418ea0071ff

═══════════ #24 ═══════════

_{Rebuilt from: notification email, session transcript. Description text is incomplete.}

When editing Sieve filters I can select "other header" & enter my own header name to filter on. However, that replaces the comparator/operation (contains/matches/…). <img width="731" height="292" alt="Image" src="https://github.com/user-attachments/assets/4caa6d13-5869-48d3-97fa-d7209552e2ec" /> - git at b194bc0a33cc2e02ef2ac53ebae03418ea0071ff ═══════════ #24 ═══════════ <sub>Rebuilt from: notification email, session transcript. Description text is incomplete.</sub>

jcoffey-dev commented

2026-08-25 15:57:11 +00:00

Owner

Fixed in #29, in main.

The condition row is a three-column grid — field, comparator, value — and the box for the header name was being rendered into the comparator's column, so the comparator disappeared along with any way to change it. The rule kept whatever it held at the moment you switched (contains, set by the field dropdown itself), which meant is and matches were unreachable for any header you typed by hand.

The header name now has a column of its own and the comparator keeps its. It also gained an aria-label — it only had a placeholder, so a screen reader had no name for it.

Thanks for the screenshot; it made the layout cause obvious straight away.

Fixed in #29, in main. The condition row is a three-column grid — field, comparator, value — and the box for the header name was being rendered into the comparator's column, so the comparator disappeared along with any way to change it. The rule kept whatever it held at the moment you switched (`contains`, set by the field dropdown itself), which meant `is` and `matches` were unreachable for any header you typed by hand. The header name now has a column of its own and the comparator keeps its. It also gained an `aria-label` — it only had a placeholder, so a screen reader had no name for it. Thanks for the screenshot; it made the layout cause obvious straight away.

mbunkus commented

2026-08-25 18:19:32 +00:00

Author

Works well; generated Sieve script looks good. Thanks!

This repo is archived. You cannot comment on issues.