Changes between Initial Version and Version 1 of Ticket #14429, comment 110


Ignore:
Timestamp:
May 8, 2015, 3:30:00 PM (5 years ago)
Author:
gk
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14429, comment 110

    initial v1  
    1111The function name might be misleading as the result can be equal to `max` as well.
    1212
    13 In `computeTargetZoom()` could you explain why you need `Math.min() there? Why not just `w - xStep` and `h- yStep` given that they are always smaller than `w` and `y` and thus the former function as a boundary. (The second `Math.min()` seems to be missing `h` as the first parameter, no?)
     13In `computeTargetZoom()` could you explain why you need `Math.min()` there? Why not just `w - xStep` and `h- yStep` given that they are always smaller than `w` and `y` and thus the former function as a boundary. (The second `Math.min()` seems to be missing `h` as the first parameter, no?)