Remove unused generic CSS classes from new Vue code
What does this MR do and why?
This removes the todo
, todo-pending
and todo-done
classes from the component.
These were not used in the Vue implementation, but only carried over from the old Haml, where they were both used for styling and test selectors.
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
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
![]() |
![]() |
How to set up and validate locally
- Enable the
todos_vue_application
feature flag. - Go to http://gdk.test:3000/dashboard/todos/vue and make sure there is no visual or behavioral change between this branch and
master
.
Related to #483055 (closed)
Edited by Thomas Hutterer
Merge request reports
Activity
Please register or sign in to reply