MaibornWolff / codecharta

CodeCharta visualizes multiple code metrics using 3D tree maps.
https://maibornwolff.github.io/codecharta/
BSD 3-Clause "New" or "Revised" License
190 stars 30 forks source link

changing display or color settings resets zoom #18

Closed silasg closed 7 years ago

silasg commented 7 years ago

when I change (at least) invert colors or toggle delta cubes, (at least) the z axis zoom is reset. however, the UI control for this zoom is not updated.

mrpatpat commented 7 years ago

When I invert the colors or toggle the delta cubes after i changed the zoom level, the z-axis zoom is not reset. Can you specify the problem and add your browser version ?

silasg commented 7 years ago

Sorry I meant y axis scaling, not z axis zoom ;) using chrome 58 on mac os, but I think I’ve also ween this using the windows electron container.


Blockchain: Euphorie oder Schockstarre – mehr in den Praxis-Tipps https://www.maibornwolff.de/newsletter/1-2017


Silas Graffy Senior IT Architect

Mobil +49 151 544 22 047 silas.graffy@maibornwolff.de Baseler Straße 10, 60329 Frankfurt am Main

MaibornWolff GmbH, Theresienhöhe 13, D-80339 München http://www.maibornwolff.de, Telefon +49 89 544 253 000 USt-ID DE 129 299 525, Amtsgericht München HRB 98058, GF: Volker Maiborn, Holger Wolff, Jens Rieger, Alexander Hofmann


On 9. Jun 2017, at 12:03, mrpatpat notifications@github.com<mailto:notifications@github.com> wrote:

When I invert the colors or toggle the delta cubes after i changed the zoom level, the z-axis zoom is not reset. Can you specify the problem and add your browser version ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/MaibornWolff/codecharta/issues/18#issuecomment-307349547, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAQzzEXStXMDKKLconUsiFuJqUfyrpHoks5sCRgmgaJpZM4NyMp8.

mrpatpat commented 7 years ago

Thank you for the contribution. I was able to reproduce this issue. I will fix it as soon as possible.

mrpatpat commented 7 years ago

fix is merged. It will be part of the next release.