-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add support for setting SheetView.showGridLines #145
Comments
I choose to implement it in another way to avoid polluting the namespace with functions for each view option:
Here is the work in progress implementing 2 options ( @xuri If this style is ok I will fully implement all options and maybe add the mirror |
Work in progress update, implementing more options ( |
Two new methods: - SetSheetViewOptions(sheetName string, viewIndex int, opts ...SheetViewOption) error - GetSheetViewOptions(sheetName string, viewIndex int, opts ...SheetViewOptionPtr) error The option values are given by the user through types that have privates methods that implement the private SheetViewOption and SheetViewOptionPtr interfaces: - DefaultGridColor(bool) - RightToLeft(bool) - ShowFormulas(bool) - ShowGridLines(bool) - ShowRowColHeaders(bool) Examples: err := xl.SetSheetViewOptions("Sheet1", -1, excelize.ShowGridLines(true)) var showGridLines excelize.ShowGridLines err := xl.GetSheetViewOptions("Sheet1", -1, &showGridLines) Fixes qax-os#145.
showGridLines
allows to disable the display of the grid.I propose to add a method
SetSheetShowGridLines(string, bool)
.I will work on a patch to implement it.
The text was updated successfully, but these errors were encountered: