Fix #2746 - Makeup causing issues aswell #2968
Conversation
app/avatar/utils.py
Outdated
| src = transform.fromfile(f'{COMPONENT_BASE}{path}') | ||
|
|
||
There was a problem hiding this comment.
W293 blank line contains whitespace
app/avatar/utils.py
Outdated
| # else: | ||
| # src_width = 900 | ||
| if src.width is not None: | ||
| src_width = float(re.sub('[^0-9]','', src.width)) |
app/avatar/utils.py
Outdated
|
|
||
| if src.height is not None: | ||
| src_height = float(re.sub('[^0-9]', '', src.height)) | ||
| else: | ||
| src_height = 1415 | ||
| scale_factor = icon_size[1] / src_height | ||
| if path.startswith('Makeup'): | ||
| scale_factor = scale_factor / 2 | ||
|
|
There was a problem hiding this comment.
W293 blank line contains whitespace
app/avatar/utils.py
Outdated
| if path.startswith('Makeup'): | ||
| x_to_center = (icon_size[0] / 2) - ((src_width * scale_factor) / 2) | ||
| svg = svg.move(x_to_center, src_height * scale_factor / 2) | ||
|
|
There was a problem hiding this comment.
W293 blank line contains whitespace
|
@kuhnchris could you resolve the stickler errors? ^_^ |
Codecov Report
@@ Coverage Diff @@
## master #2968 +/- ##
==========================================
- Coverage 29.78% 29.72% -0.07%
==========================================
Files 180 180
Lines 13720 13728 +8
Branches 1839 1842 +3
==========================================
- Hits 4086 4080 -6
- Misses 9496 9510 +14
Partials 138 138
Continue to review full report at Codecov.
|
|
Do I have to? Awww. 😭 Anyways, fix inbound! |
|
see #2746 (comment)