Move to-dos actions out of the main link

What does this MR do and why?

Move to-dos actions out of the main link

Moving actions out of the main to-do link ensures that they can be interacted with without event listeners from conflicting with each other. Specifically, this fixes a bug where the snoozing dropdown doesn't work on Firefox as clicking on any option triggers the outside directive, effectively closing the dropdown panel without activating the clicked option.

This also improves the HTML validity as no interactive elements should be rendered within links.

Changelog: fixed

References

Please include cross links to any resources that are relevant to this MR. This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Desktop

Before After
Screenshot_2025-01-14_at_1.47.01_PM Screenshot_2025-01-14_at_1.45.32_PM

Mobile

Before After
Screenshot_2025-01-14_at_1.46.48_PM Screenshot_2025-01-14_at_1.53.21_PM

Snoozing a to-do on Firefox

Before After
bug fixed

How to set up and validate locally

  1. Enable the todos_snoozing feature flag: http://gdk.test:3000/rails/features/todos_snoozing / https://gdk.test:3443/rails/features/todos_snoozing.
  2. On Firefox, navigate to the to dos dashboard: https://gdk.test:3443/dashboard/todos.
  3. Snooze some to-do items using the disclosure dropdown.

Related to #512900 (closed)

Edited by Paul Gascou-Vaillancourt

Merge request reports

Loading