Skip to content

DM-9691: time series image problems fixed #328

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

Merged
merged 1 commit into from
Mar 5, 2017
Merged

Conversation

robyww
Copy link
Contributor

@robyww robyww commented Mar 5, 2017

  • expanded mode problem fixed (flex layout issue)
  • image zoom levels sizes stay correctly syncronized
  • images reset when new data is loaded
  • images reset when cutout size changes and stay synced
  • when rotation happens on images, target match is turned off. Therefore new
    images will not come in at the same rotation. The target match
    must be turned back on. This is the expected behavior we can consider
    whether is is the desired behavior.

 - expanded mode problem fixed (flex layout issue)
 - image zoom levels sizes stay correctly syncronized
 - images reset when new data is loaded
 - images reset when cutout size changes and stay synced
 - when rotation happens on images, target match is turned off. Therefore new
   images will not come in at the same rotation.  The target match
   must be turned back on. This is the expected behavior we can consider
   whether is is the desired behavior.
@robyww robyww self-assigned this Mar 5, 2017
@robyww robyww requested a review from tgoldina March 5, 2017 00:46
Copy link
Contributor

@tgoldina tgoldina left a comment

Choose a reason for hiding this comment

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

All items checked. Works as expected.

@robyww robyww merged commit 31dbb68 into dev Mar 5, 2017
@robyww robyww deleted the dm-9691-image-issues branch March 5, 2017 23: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