Bug scroll to viewport

[tung13041996]

This pull request to fixed bug: click to item accordion, window will scroll to viewport.

[phucbm]

I don't understand why "scroll to viewport" is a bug. Could you please explain more? In what case do we need this?

[phucbm]

@tung13041996
Any update on this?

[tung13041996]

@tung13041996 Any update on this?

I will update code for this issue because of bug in other project when update to newest version of ETA

[phucbm]

@tung13041996

I don't understand why "scroll to viewport" is a bug. Could you please explain more? In what case do we need this?

1. You have not answer this question.

@tung13041996 Any update on this?

I will update code for this issue because of bug in other project when update to newest version of ETA

2. I ask for update because I want to know why did we not close this issue. Your answer did not help me to make any decision, and I think that you are prolonging. At least give me a deadline.

[vuquangpham]

About the scrollIntoView method, we make the viewport jump to the wrapper of the target, that's wrong position,

[tung13041996]

@phucbm First, I so sorry for missing notification for this issue. Then, I'll explain why it's a bug.
In all project, trigger is <a> with href=#...., so function assignTriggerElements in methods.js will run. So, when we click trigger, line 66 will run and scroll to wrapper. That's the reason I remove line 66.

[phucbm]

Thank you @tung13041996, it's clear now.

[phucbm]

LGTM.