Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve entity tags and permissions table layout #1021

Merged
merged 3 commits into from
Oct 16, 2018

Conversation

swaterkamp
Copy link
Member

  • Center Actions column
  • Remove toggleDetailsIcon from permissions tab

@swaterkamp swaterkamp self-assigned this Oct 12, 2018
/>
</IconDivider>
<TableData align="center">
<span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the span really necessary? At least it shouldn't.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It shouldn't, but it is. The IconDivider was not centered without the span.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try setting the align at the IconDivider?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I did, but it didn't work. What I did not test, though, is to set the IconDivider to grow. A growing divider plus the align="center" in its tag solves the issue.

However: The margin on top of the table does not collapse and it seem 
like there is an invisible pagination component somewhere. Needs to be 
found and fixed in the future.
@codecov
Copy link

codecov bot commented Oct 16, 2018

Codecov Report

Merging #1021 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1021      +/-   ##
=========================================
+ Coverage     9.5%   9.51%   +0.01%     
=========================================
  Files         820     820              
  Lines       26650   26615      -35     
  Branches     5621    5606      -15     
=========================================
  Hits         2533    2533              
+ Misses      22499   22466      -33     
+ Partials     1618    1616       -2
Impacted Files Coverage Δ
gsa/src/web/entity/permissions.js 0% <ø> (ø) ⬆️
gsa/src/web/entity/tags.js 0% <ø> (ø) ⬆️
gsa/src/web/pages/start/page.js 0% <0%> (ø) ⬆️
gsa/src/web/components/chart/line.js 0% <0%> (ø) ⬆️
gsa/src/web/components/dashboard/dashboard.js 0% <0%> (ø) ⬆️
gsa/src/web/pages/start/dashboard.js 0% <0%> (ø) ⬆️
...rc/web/components/dashboard/display/datadisplay.js 0% <0%> (ø) ⬆️

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 bdb10c1...14d62fe. Read the comment docs.

@bjoernricks bjoernricks self-requested a review October 16, 2018 08:47
@bjoernricks bjoernricks merged commit fe48754 into greenbone:master Oct 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants