-
Notifications
You must be signed in to change notification settings - Fork 137
create a component to better handle tabular data #1
Comments
btw long term this should replace browser-laptop SortableTable component |
not sure what advantage we get from using css grid for a table, but I'd suggest React Virtualized for a popular component that can handle thousands of items and render only what's visible. It has a lot of different types of components, helpers and very easily styled. You can use it for lists not just tables, and so put your own completely custom components in. |
reason for grid is to make use of ya React Virtualized is the strongest option by now. |
by the way, you can use position: sticky on a table, just need to do it on
all the TH cells, not the whole THEAD
…On Thu, Nov 2, 2017 at 7:23 PM Cezar Augusto ***@***.***> wrote:
reason for grid is to make use of position: sticky mainly, could be made
by other means btw like flexbox or any other positioning of choice just
want to avoid table.
ya React Virtualized is the strongest option by now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAtRzNGbPToK8lQ7FdTx_lrQ2U846hrGks5synkogaJpZM4QNtH9>
.
|
after you mentioned I got really into this and found https://bugs.chromium.org/p/chromium/issues/detail?id=702927 so that's possible as you said: https://jsfiddle.net/cezaraugusto/ww6237cL/ which is confusing, iirc it's per spec not supposed to work https://drafts.csswg.org/css-position/#position-property and https://drafts.csswg.org/css-position/#valdef-position-sticky. For me it's still hacky but well we can use table thanks Google 🙂 |
Yeah I know all the hacks :-) well l, not really, just that one because
I’ve been subscribed to all the position sticky bugs waiting for it to
finally happen
…On Thu, Nov 2, 2017 at 10:31 PM Cezar Augusto ***@***.***> wrote:
after you mentioned I got really into this and found
https://bugs.chromium.org/p/chromium/issues/detail?id=702927 so that's
possible as you said: https://jsfiddle.net/cezaraugusto/ww6237cL/
which is confusing, iirc it's per spec not supposed to work
https://drafts.csswg.org/css-position/#position-property and
https://drafts.csswg.org/css-position/#valdef-position-sticky. For me
it's still hacky but well we can use table thanks Google 🙂
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#1 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAtRzIuaKfgQd102v83bCPZfMDZPf7wHks5syqUVgaJpZM4QNtH9>
.
|
we have it now so closing |
follow-up of brave/browser-laptop#9032. We need this to unblock brave/browser-laptop#11177 and fix brave/browser-laptop#11551. This component should be prioritized over others.
Component should:
The text was updated successfully, but these errors were encountered: