calamine icon indicating copy to clipboard operation
calamine copied to clipboard

feat: Implement RichText support for XLSX

Open FlixCoder opened this issue 1 year ago • 2 comments

For #440 .

I implemented a RichText representation and changed the Data representation to use it. The parsing is so far only implemented for XLSX, the other formats continue to only ever produce Data::String even if it is actually rich text content.

The shared strings also have to be rich text now ^^

I think it turned out quite well, let me know what you think :)

FlixCoder avatar Jun 06 '24 11:06 FlixCoder

Adjusted to the requested changes. I also found a few more little things that I missed, e.g. the DataType::get_string implementation for Data and such.

FlixCoder avatar Jun 07 '24 09:06 FlixCoder

I can't really progress until https://github.com/tafia/calamine/pull/441#discussion_r1633001639 . Could you elaborate a bit how you imagine things to be?

FlixCoder avatar Jun 14 '24 16:06 FlixCoder