Skip to content

fix(account): filter buttons not working (@fehmer)#7505

Merged
Miodec merged 2 commits intomasterfrom
feature/fix-resultfiter-chart
Feb 28, 2026
Merged

fix(account): filter buttons not working (@fehmer)#7505
Miodec merged 2 commits intomasterfrom
feature/fix-resultfiter-chart

Conversation

@fehmer
Copy link
Copy Markdown
Member

@fehmer fehmer commented Feb 15, 2026

unable to select filter presets, chart not updating

reproduce:

  • create two presets e.g. all and last three month
  • switch between presets

The current preset doesnt get highlighted and the chart doesnt get updated.

advanced filters

  • clear filters not working

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Feb 15, 2026
@fehmer fehmer force-pushed the feature/fix-resultfiter-chart branch from 9c1577a to 42cbc62 Compare February 15, 2026 12:21
@fehmer fehmer changed the title fix(account): usable to select presets, chart not updating (@fehmer) fix(account): usable to select filter presets, chart not updating (@fehmer) Feb 15, 2026
@fehmer fehmer changed the title fix(account): usable to select filter presets, chart not updating (@fehmer) fix(account): filter buttons not working (@fehmer) Feb 17, 2026
toggle(group, filter);
// filters[group][filter] = !filters[group][filter];
} else {
const group = (e.target as HTMLElement).parentElement?.getAttribute(
Copy link
Copy Markdown
Contributor

@Leonabcd123 Leonabcd123 Feb 19, 2026

Choose a reason for hiding this comment

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

Can we use the already existing variable childTarget instead of e.target?

@Miodec Miodec merged commit 427e9de into master Feb 28, 2026
20 checks passed
@Miodec Miodec deleted the feature/fix-resultfiter-chart branch February 28, 2026 14:37
IliyaZinoviev pushed a commit to IliyaZinoviev/monkeytype that referenced this pull request Apr 24, 2026
## unable to select filter presets, chart not updating

reproduce:

- create two presets e.g. `all` and `last three month`
- switch between presets

The current preset doesnt get highlighted and the chart doesnt get
updated.

## advanced filters 
- clear filters not working
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants