Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Léo-Paul Géneau
gitlab-ce
Commits
d45a8e00
Commit
d45a8e00
authored
Jul 11, 2017
by
Fatih Acet
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
IssueNotesRefactor: Make scroll into note an action.
parent
08d597c7
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
8 additions
and
5 deletions
+8
-5
app/assets/javascripts/notes/components/issue_notes.vue
app/assets/javascripts/notes/components/issue_notes.vue
+1
-5
app/assets/javascripts/notes/stores/issue_notes_store.js
app/assets/javascripts/notes/stores/issue_notes_store.js
+7
-0
No files found.
app/assets/javascripts/notes/components/issue_notes.vue
View file @
d45a8e00
...
...
@@ -94,12 +94,8 @@ export default {
const
$el
=
$
(
`#
${
hash
}
`
);
if
(
hash
&&
$el
)
{
const
isInViewport
=
gl
.
utils
.
isInViewport
(
$el
[
0
]);
this
.
$store
.
commit
(
'
setTargetNoteHash
'
,
hash
);
if
(
!
isInViewport
)
{
gl
.
utils
.
scrollToElement
(
$el
);
}
this
.
$store
.
dispatch
(
'
scrollToNoteIfNeeded
'
,
$el
);
}
},
},
...
...
app/assets/javascripts/notes/stores/issue_notes_store.js
View file @
d45a8e00
...
...
@@ -214,6 +214,13 @@ const actions = {
}
});
},
scrollToNoteIfNeeded
(
context
,
el
)
{
const
isInViewport
=
gl
.
utils
.
isInViewport
(
el
[
0
]);
if
(
!
isInViewport
)
{
gl
.
utils
.
scrollToElement
(
el
);
}
},
};
export
default
{
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment