This repository has been archived by the owner on Mar 2, 2018. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 45
Buggy diagram width/height #31
Comments
Hello, thanks for your report and I'm very very sorry for late reply. I've not realized this ticket so far. I've seen some of these issues and I'd like to fix them, but unfortunately I don't have a time to improve because I'm busy on my daily biz... It would be thankful for me if you are waiting for fix patiently. Thanks! |
No problem. I can live with those tiny glitches. :) |
@akfish - this is awesome bug reporting! I'd vote to split them up into their own individual issues so they can be tackled independently/prioritized. |
JFYI: I think case 2 & case 3 were fixed by 6d6e17c or other commits. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Test Setup
Bugs described in this issue are usually hard to notice. I stumped upon them when using style:
All tests are done in Chrome 33
Case 1 - Problem with padding
This happens with robustness diagram.
Code:
Result when padding: 10px
The behavior is inconsistent in different cases:
Case 2 - Problem with shadows
No padding:
As you can see, sequence diagram and robustness diagram have different behaviors. The former one didn't take account of shadows when calculating width.
Case 3 - Problem with custom styling
Code:
No padding:
It looks like that the size is calculated before custom styling is done.
The text was updated successfully, but these errors were encountered: