Skip to content

Add refresh button to description page#771

Merged
RMacfarlane merged 1 commit intomasterfrom
rmacfarlane/refresh
Jan 4, 2019
Merged

Add refresh button to description page#771
RMacfarlane merged 1 commit intomasterfrom
rmacfarlane/refresh

Conversation

@RMacfarlane
Copy link
Contributor

@RMacfarlane RMacfarlane commented Jan 4, 2019

Fixes #103

Copy link
Member

@rebornix rebornix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks good to me. I found that the set-scroll in update doesn't work as expected somehow but it's unrelated to this pr.

		this._postMessage({
			command: 'set-scroll',
			scrollPosition: this._scrollPosition,
		});

@RMacfarlane
Copy link
Contributor Author

what's the behavior you're seeing for set-scroll? Let's file a bug for that

@RMacfarlane RMacfarlane merged commit dc1df81 into master Jan 4, 2019
@RMacfarlane RMacfarlane deleted the rmacfarlane/refresh branch January 4, 2019 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants