Skip to content
Snippets Groups Projects
Commit 3fcc78d6 authored by DemetriadS's avatar DemetriadS
Browse files

fix(ControlledAccordion): Invite HE auto-scroll accordion under header

parent c9a4bf06
No related branches found
No related tags found
2 merge requests!110Sprint 21 Features,!69fix(ControlledAccordion): Invite HE auto-scroll accordion under header
...@@ -17,11 +17,17 @@ const HeaderComponent = ({ icon, label, toggle, expanded, ...props }) => ( ...@@ -17,11 +17,17 @@ const HeaderComponent = ({ icon, label, toggle, expanded, ...props }) => (
) )
class ControlledAccordion extends React.Component { class ControlledAccordion extends React.Component {
componentDidMount() {
this.scroller = document.querySelector('.faraday-root')
}
componentDidUpdate(prevProps) { componentDidUpdate(prevProps) {
const shouldScroll = !prevProps.expanded && this.props.expanded const shouldScroll = !prevProps.expanded && this.props.expanded
if (this.props.scrollIntoView && shouldScroll) { if (this.props.scrollIntoView && shouldScroll) {
this._accordion.scrollIntoView && this._accordion.scrollIntoView() const appBarHeight = 70 // TODO -- take height from constants
const appBarMargin = 16 // TODO -- take margin from constants
this.scroller.scrollTop =
this._accordion.offsetTop - appBarHeight - appBarMargin
} }
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment