Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Issue #37 : On editing a post, scroll partial to top of preview. #220

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Issue #37 : On editing a post, scroll partial to top of preview. #220

wants to merge 3 commits into from

Commits on Aug 16, 2016

  1. Issue #37 : On editing a post, scroll partial to top of preview. 

    Editing a post field will trigger a refresh of its partial.
In the refresh() method, define and call scrollPartialToTopOfPage.
Some themes have a margin-top of the #page element, like 2016.
And this scrolling would hide part of the partial.
So also calculate the offsetTop of #page, to prevent this.
If this isn't a concern, this commit could be much simpler.
    Ryan Kienstra committed Aug 16, 2016
    Configuration menu
    Copy the full SHA
    75f1694 View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2016

  1. Issue #37 : Remove extra parentheses to avoid linting error.

    Ryan Kienstra committed Aug 17, 2016
    Configuration menu
    Copy the full SHA
    9861a39 View commit details
    Browse the repository at this point in the history

  2. Issue #37 : Revert previous 2 commits to take different approach. 

    Revert 9861a39 and 75f1694.
    Ryan Kienstra committed Aug 17, 2016
    Configuration menu
    Copy the full SHA
    da67beb View commit details
    Browse the repository at this point in the history