raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-01-24 20:36:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove need for class

Browse Source 
Generally, I would say that style rules should have a BEM-style class name, but in this case it's terribly convenient to support a tags directly.
This commit is contained in:
David Cook
2023-11-24 16:14:28 +11:00
parent c6fc139f62
commit 2bd2bea7b7
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/views/admin/products_v3/components/_product_actions.html.haml
View File

@@ -2,7 +2,7 @@
%i.fa.fa-ellipsis-v{ "data-action": "click->vertical-ellipsis-menu#toggle" }
.vertical-ellipsis-menu-content{ "data-vertical-ellipsis-menu-target": "content" }
- if defined?(variant)
= link_to t('admin.products_page.actions.edit'), edit_admin_product_variant_path(product, variant), class: "vertical-ellipsis-menu-content-item"
= link_to t('admin.products_page.actions.edit'), edit_admin_product_variant_path(product, variant)
- else
= link_to t('admin.products_page.actions.edit'), edit_admin_product_path(product), class: "vertical-ellipsis-menu-content-item"
= link_to t('admin.products_page.actions.clone'), clone_admin_product_path(product), class: "vertical-ellipsis-menu-content-item"
= link_to t('admin.products_page.actions.edit'), edit_admin_product_path(product)
= link_to t('admin.products_page.actions.clone'), clone_admin_product_path(product)

2
app/webpacker/css/admin_v3/components/vertical_ellipsis_menu.scss
View File

@@ -30,7 +30,7 @@
display: block;
}
.vertical-ellipsis-menu-content-item {
& > a {
display: block;
padding: 5px 10px;
cursor: pointer;