sebawagner commented on a change in pull request #75:
URL: https://github.com/apache/openmeetings/pull/75#discussion_r420483176
##########
File path: openmeetings-web/src/main/webapp/css/raw-tree.css
##########
@@ -24,6 +23,90 @@
.file-tree .footer .sizes .size {
padding-left: 5px;
}
+/* Calc width with default for max level of 7 */
+.file-tree .file.item .name
+ , .file-tree .file.item .name span
+{
+ max-width: calc(var(--room-sidebar-width) - 26px);
+ min-width: 60px;
+}
+.file-tree .tree-subtree .file.item .name
Review comment:
min-width: 60px;
is already applied to every node. Its on the generic CSS rule that all tree
nodes should have. So. I removed it and not repeat it into every subsequent
rule for each tree level.
overflow-x: auto
I would need to rethink. Whats the point in calculating max-width size for
each line item, if you scroll the entire panel anyway ? I thought the purpose
is to have the warning icon visible.
But yeah, probably you can still add a scrollbar horizontally, it won't be
possible to avoid scrolling at some stage. And I think most users will expect
to be able to scroll if its outside the visible area.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
