Skip to content
This repository has been archived by the owner. It is now read-only.

Scroll on actions #1284

Closed
edmundoa opened this issue May 18, 2015 · 2 comments
Closed

Scroll on actions #1284

edmundoa opened this issue May 18, 2015 · 2 comments
Assignees
Milestone

Comments

@edmundoa
Copy link
Member

@edmundoa edmundoa commented May 18, 2015

When clicking on the "add to search" button, adding statistics, adding field graphs, or adding quick values, we should show some indication that we added some elements into de page, and maybe scroll to them.

@edmundoa edmundoa added this to the 1.1.0 milestone May 18, 2015
@edmundoa edmundoa self-assigned this May 28, 2015
edmundoa added a commit that referenced this issue May 28, 2015
edmundoa added a commit that referenced this issue May 28, 2015
edmundoa added a commit that referenced this issue May 28, 2015
We need to use a different hint element to avoid conflicts with main.js

Refs #1284
edmundoa added a commit that referenced this issue May 28, 2015
Otherwise the user may get scroll hints for all field graphs when the
page is reloaded.

That implies that we need to fix the way we handled the events from the
old fieldcharts.js

Refs #1284
@edmundoa
Copy link
Member Author

@edmundoa edmundoa commented May 28, 2015

Fixed with cb57495, 2ced1d7, 0136e8b, 71559b8, and e053fd0.

@edmundoa edmundoa closed this May 28, 2015
@kroepke
Copy link
Contributor

@kroepke kroepke commented May 28, 2015

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants