From 848c245db6d0da1505a80907145166eb38b11334 Mon Sep 17 00:00:00 2001 From: Eike Kettner Date: Mon, 9 Nov 2020 14:36:58 +0100 Subject: [PATCH] Change the card link to only use the main content The card is no longer a link itself. The main target is moved to be the content (the area containing the title and tags). This is in preparation of upcoming changes: if the whole card is a link, it cannot contain other links, due to a restriction by html. Later a card may have more links to provide. --- modules/webapp/src/main/elm/Comp/ItemCardList.elm | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/modules/webapp/src/main/elm/Comp/ItemCardList.elm b/modules/webapp/src/main/elm/Comp/ItemCardList.elm index 2b6fd4fc..675847b3 100644 --- a/modules/webapp/src/main/elm/Comp/ItemCardList.elm +++ b/modules/webapp/src/main/elm/Comp/ItemCardList.elm @@ -219,15 +219,13 @@ viewItem cfg settings item = Data.ItemSelection.Active ids -> onClick (ToggleSelectItem ids item.id) in - a + div ([ classList [ ( "ui fluid card", True ) , ( cardColor, True ) , ( "current", cfg.current == Just item.id ) ] , id item.id - , href "#" - , cardAction ] ++ DD.draggable ItemDDMsg item.id ) @@ -243,7 +241,11 @@ viewItem cfg settings item = ] [] ] - , div [ class "content" ] + , a + [ class "content" + , href "#" + , cardAction + ] [ case cfg.selection of Data.ItemSelection.Active ids -> div [ class "header" ]