diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.css index d39cbe2..681e6dd 100644 --- a/app/assets/stylesheets/application.css +++ b/app/assets/stylesheets/application.css @@ -95,9 +95,11 @@ button > svg { padding-right: 0.4em; width: 1.8em; } -/* TODO: hover/focus here and remove from table.items */ -/* This is for normal links only. Button-like link styles are under .button - * selector */ + +/* TODO: move normal non-button links (:hover/:focus) styling here (i.e. + * page-wide, top-level) and remove from table.items - as the style should be + * same everywhere */ + .button:focus-visible, button:focus-visible, input[type=submit]:focus-visible { @@ -339,7 +341,7 @@ table.items td.actions { justify-content: end; } -/* TODO: replace :hover:focus-visible with proper stye order */ +/* TODO: replace :hover:focus-visible combos with proper LOVE stye order */ table.items td.link a:hover, table.items td.link a:focus-visible, table.items td.link a:hover:focus-visible { diff --git a/test/system/users_test.rb b/test/system/users_test.rb index d615481..8a59e71 100644 --- a/test/system/users_test.rb +++ b/test/system/users_test.rb @@ -110,8 +110,8 @@ class UsersTest < ApplicationSystemTestCase click_on t("layouts.application.users") all(:link_or_button, text: t("users.index.disguise")).sample.click assert_current_path edit_user_registration_path - # TODO: test for profile app-menu link after root changed to different path - # then profile + # TODO: test for "Profile" link button after root_url changed to different + # path then profile in routes.rb click_on t("layouts.application.revert") assert_current_path users_path @@ -134,10 +134,11 @@ class UsersTest < ApplicationSystemTestCase assert_title "Bad request received (400)" end - test "destroy profile" do + test "delete profile" do user = users.select(&:confirmed?).sample sign_in user: user - # TODO: remove condition after root changed to different path than profile + # TODO: remove condition after root_url changed to different path than + # profile in routes.rb unless has_current_path?(edit_user_registration_path) first(:link_or_button, user.email).click end