Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scroll doesn't work in the full screen mode #12832

Closed
Basts opened this issue Jan 22, 2020 · 0 comments · Fixed by #13172
Closed

Scroll doesn't work in the full screen mode #12832

Basts opened this issue Jan 22, 2020 · 0 comments · Fixed by #13172
Assignees

Comments

@Basts
Copy link
Contributor

Basts commented Jan 22, 2020

Scroll disappears in the full-screen mode.

Live demo with steps to reproduce

Vertical scroll demo
Horizontal scroll demo

Choose the 'view in full screen' in the export menu to see the bug.

Product version

Highcharts JS v8.0.0 (2019-12-10)

Internal note:

It works fine after changes in the code and back to previous state.
Should be clearly shown here: https://imgur.com/a/saeKSbL

------ EDIT -------

scrollablePlotArea changes chart DOM and separate some parts to different divs like:

<div id="container">
   <div id="highcharts-....." class="highcharts-container">
   </div>
</div>

to:

<div id="container">
   <div class="highcharts-fixed">
   </div>
   <div class="highcharts-scrolling">
      <div class="highcharts-inner-container">
          <div id="highcharts-....." class="highcharts-container">
          </div>
      </div>
   </div>
</div>

Current Fullscreen.prototype.open implementation affects only on the "highcharts-inner-container". There will be needed some additional condition to affect on other parents when scrollable is applied.

Current workaround only for the vertical scroll (description inside): https://jsfiddle.net/BlackLabel/1pvLn49e/

@highsoft-bot highsoft-bot added this to To do in Development-Flow Mar 5, 2020
@Basts Basts self-assigned this Mar 19, 2020
@highsoft-bot highsoft-bot moved this from To do to In progress in Development-Flow Mar 20, 2020
Basts pushed a commit that referenced this issue Mar 20, 2020
Basts pushed a commit that referenced this issue Mar 20, 2020
@highsoft-bot highsoft-bot moved this from In progress to To do in Development-Flow Mar 20, 2020
Development-Flow automation moved this from To do to Done Mar 30, 2020
TorsteinHonsi pushed a commit that referenced this issue Mar 30, 2020
TorsteinHonsi pushed a commit that referenced this issue Mar 30, 2020
@Izothep Izothep removed this from Done in Development-Flow Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants