Skip to content

Commit fe3aa92

Browse files
committed
Fix scroll to from diff sidebarg
1 parent 4ffc9c8 commit fe3aa92

File tree

1 file changed

+21
-2
lines changed

1 file changed

+21
-2
lines changed

src/UnisonShare/Page/ProjectContributionChangesPage.elm

Lines changed: 21 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ type Msg
8282
| RetryBranchDiffFetch Int
8383
| ToggleChangeDetails ChangeLine
8484
| CopyChangeLinePermalink ChangeLineId
85+
| SetChangeLinePermalink ChangeLineId
8586
| ScrollTo ChangeLineId
8687
| NoOp
8788

@@ -159,6 +160,19 @@ update appContext projectRef contribRef msg model =
159160
_ ->
160161
( model, Cmd.none )
161162

163+
SetChangeLinePermalink changeLineId ->
164+
let
165+
route =
166+
changeLineId
167+
|> Route.projectContributionChange projectRef contribRef
168+
in
169+
( model
170+
, Cmd.batch
171+
[ Route.navigate appContext.navKey route
172+
, scrollTo changeLineId
173+
]
174+
)
175+
162176
CopyChangeLinePermalink changeLineId ->
163177
let
164178
route =
@@ -315,8 +329,13 @@ viewDiffTreeNode : ProjectRef -> ToggledChangeLines -> ChangeLine -> Html Msg
315329
viewDiffTreeNode projectRef toggledChangeLines changeLine =
316330
let
317331
viewTitle fqn =
318-
Click.onClick (ToggleChangeDetails changeLine)
319-
|> Click.view [ class "change-title" ] [ FQN.view fqn ]
332+
case ChangeLine.toChangeLineId changeLine of
333+
Just changeLineId ->
334+
Click.onClick (SetChangeLinePermalink changeLineId)
335+
|> Click.view [ class "change-title" ] [ FQN.view fqn ]
336+
337+
Nothing ->
338+
div [ class "change-title" ] [ FQN.view fqn ]
320339

321340
view_ type_ content =
322341
div [ class "change-line" ]

0 commit comments

Comments
 (0)