substance / texture

A visual editor for research.
MIT License
1k stars 85 forks source link

Ui improvements #1191

Closed Integral closed 5 years ago

Integral commented 5 years ago

As suggested in #1189.

obuchtala commented 5 years ago

This was more massive than we thought this morning.

obuchtala commented 5 years ago

But I am very proud of it now :)

codecov[bot] commented 5 years ago

Codecov Report

Merging #1191 into master will increase coverage by 0.03%. The diff coverage is 88.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1191      +/-   ##
==========================================
+ Coverage   86.35%   86.38%   +0.03%     
==========================================
  Files         398      398              
  Lines        8946     8917      -29     
==========================================
- Hits         7725     7703      -22     
+ Misses       1221     1214       -7
Impacted Files Coverage Δ
src/article/shared/TableContextMenu.js 100% <ø> (ø) :arrow_up:
src/kit/ui/ToolPrompt.js 100% <ø> (ø) :arrow_up:
src/kit/ui/Toolbar.js 100% <ø> (ø) :arrow_up:
src/article/shared/xrefHelpers.js 87.5% <ø> (ø) :arrow_up:
src/kit/ui/ToggleTool.js 0% <0%> (-100%) :arrow_down:
src/kit/ui/ContextMenu.js 45% <0%> (+1.52%) :arrow_up:
src/TextureConfigurator.js 90.9% <0%> (+5.38%) :arrow_up:
src/article/shared/ArticleToolbarPackage.js 100% <100%> (ø) :arrow_up:
src/article/ArticleAPI.js 97.05% <100%> (ø) :arrow_up:
src/article/ArticlePanel.js 86.48% <100%> (+0.57%) :arrow_up:
... and 30 more

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 6567f9e...1d5d822. Read the comment docs.

obuchtala commented 5 years ago

@Integral how do you like this colour? ;)

obuchtala commented 5 years ago

@Integral everything seems to be working. Now we need to fix styles. And also still left is hiding inactive groups from the 'Edit' dropdown.

Integral commented 5 years ago

👍

Integral commented 5 years ago

I would say it is now in a good shape, but want to discuss one tiny detail.