Closed cjmayo closed 3 years ago
Merging #233 (4024d14) into master (8e103be) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #233 +/- ##
=======================================
Coverage 85.94% 85.94%
=======================================
Files 55 55
Lines 10388 10390 +2
=======================================
+ Hits 8928 8930 +2
Misses 1460 1460
Impacted Files | Coverage Δ | |
---|---|---|
fract4dgui/angle.py | 83.72% <100.00%> (ø) |
|
fract4dgui/application_widgets.py | 91.80% <100.00%> (+0.06%) |
:arrow_up: |
fract4dgui/application_window.py | 89.23% <100.00%> (+0.04%) |
:arrow_up: |
fract4dgui/fourway.py | 93.82% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 8e103be...d23733b. Read the comment docs.
With a 1024x768 fractal I had a blank margin down the right of the application, changing to setting margin and padding for the toolbar in CSS fixed this for me. It is also what is recommended and available in GTK 4.
The Preview fractal has more space below than above, this can be seen on macOS: https://user-images.githubusercontent.com/1893197/102902879-72023680-4467-11eb-9541-4be960fad93b.png
Increasing the size of the Preview to 53 pixels fixes this, make this one value used by all of the toolbar items. This is just a constant here, but could be the start of making it configurable in the future.