Closed sravan-s closed 7 years ago
Hi!
Interesting suggestion – I have a few questions about the implementation. Can the ellipsis be set directly to <data-table-cell>
so that an extra <span>
isn't needed? If yes, I think this could be set as a default style so that a property ellipsis
is not needed.
Also, I'd leave out setting any values to the title
; I'm worried making assumptions that innerHTML
contains the correct value (it might have HTML).
@Saulis I'll try out your suggestion on putting style directly on data-table-cell. I really need that text on hover. Any ideas for implementing it? Actually, I put the 'span' inside data-table-cell to implement that text over hover.
Well, I checked it. But ellipsis seems to have some problems with display: flex. We are using dispaly: flex on data-table-cell. Wrapping text inside a container seems to help.
Would like to have an inbuilt property to handle overflown text, as in here:
I have implemented a fix in my branch: feature/cell-overflow-ellipsis. The results are something like this.
If you guys think this is a valid request, I'll raise a pull request.