Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Actual Excel Themes #6

Open
myoder020 opened this issue May 2, 2024 · 7 comments
Open

Actual Excel Themes #6

myoder020 opened this issue May 2, 2024 · 7 comments

Comments

@myoder020
Copy link

Any plans on implementing the theming we get in Excel, like the actual table styles?
I tried applying a theme using the cellStyle but the theme index number isn't the same as what we get in Excel so I'm having to guess.

Also any way to enable filtering on a column?

@ChronicStone
Copy link
Owner

ChronicStone commented May 2, 2024

Hi @myoder020 regarding excel themes, I'm not sure, the library uses xlsx-js-style, a fork of SheetJs supporting cell styling internally, so you should look into their docs / issues to see if that's possible.

Regarding column filtering, I'm not sure what you mean by that. Can you give me a bit more details, and explain what you're trying to achieve / show what kind of API you would expect?

@myoder020
Copy link
Author

Filter:

image

Just trying to do something that you can do in a normal excel worksheet

@ChronicStone
Copy link
Owner

ChronicStone commented May 2, 2024

I think that might be possible, just checked SheetJs docs and looks like it supports autoFilter. I have a bit of spare time in about 30 minutes, I'll try to give it a shot.

I have a few concerns, seems possible only for mono-tables sheets and I don't know how it will behave with merge cells.

If you have an example excel file I can use for reference that'd help a lot @myoder020

@myoder020
Copy link
Author

My use case is just for basic reports.
They don't involve merged cells or sheets with multiple tables.

@ChronicStone
Copy link
Owner

@myoder020 Yeah but I need to make sure this does not break the library on more complex cases. The basic use case seem to work, I just need to fine tune a few things
image

@ChronicStone
Copy link
Owner

That feature got a few caveats, I'll need to polish this, will try to ship it before the end of the week if I can find workaround for the edge cases

@myoder020
Copy link
Author

Thanks for looking into it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants