create a component to better handle tabular data

[cezaraugusto]

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:

• make use of css grid
• support large amounts of data (such as cookies, browser history, etc)

[cezaraugusto]

btw long term this should replace browser-laptop SortableTable component

[petemill]

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.

[cezaraugusto]

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.

[petemill]

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> .

[cezaraugusto]

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 🙂

[petemill]

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> .

[cezaraugusto]

we have it now so closing