1. OpenREM
  2. OpenREM
  3. OpenREM
  4. Issues

Issues

Issue #306 resolved

Add a button to each chart to make it full-screen

David Platten
created an issue

I would find it useful to be able to view the charts full-screen sometimes.

Comments (17)

  1. David Platten reporter

    I do plan to implement this for CT too. I think that some of my JavaScript is Chrome-specific at the moment - it needs to be checked on some other browsers before I sort out the CT charts too.

  2. Ed McDonagh

    Ok. If there is any possibility it won't make the cut (and perhaps anyway, just for good practice), can you fork it into a new branch? Else we might need to try and extract it to make a release...

    Thanks.

  3. David Platten reporter

    Added full screen buttons to CT charts. Moved JavaScript into external file. Tested and works with Chrome. Edge and Firefox also work, but charts don't revert back to their original height when exiting full screen view. References issue #306

    → <<cset 68ce7c8eabcd>>

  4. David Platten reporter

    Charts now return to their correct height once exiting full-screen mode. Tested with Chrome 46 and Internet Explorer 8. I'll test on other browsers at home, and if they work OK I'll close the issue. References issue #306.

    → <<cset f7bd50cb72d8>>

  5. Ed McDonagh

    Lovely. Only looked in Chrome at the moment, but lovely.

    Different issue, but can you remove the inconsistency that opening I think the second chart closes the first, concertina style, but the others don't?

  6. Log in to comment