Merge pull request qax-os#156 from dolmen-go/fix/SheetViewOptPtr-is-a…

…-SheetViewOpt SheetViewOptionPtr: document that it is a superset of SheetViewOption
dolfly · Nov 21, 2017 · 3d231cb · 3d231cb
2 parents 130eb45 + d2fb019
commit 3d231cb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/sheetview.go b/sheetview.go
@@ -9,6 +9,7 @@ type SheetViewOption interface {
 
 // SheetViewOptionPtr is a writable SheetViewOption. See GetSheetViewOptions().
 type SheetViewOptionPtr interface {
+	SheetViewOption
 	getSheetViewOption(view *xlsxSheetView)
 }
 

diff --git a/sheetview_test.go b/sheetview_test.go
@@ -13,6 +13,12 @@ var _ = []excelize.SheetViewOption{
 	excelize.ShowFormulas(false),
 	excelize.ShowGridLines(true),
 	excelize.ShowRowColHeaders(true),
+	// SheetViewOptionPtr are also SheetViewOption
+	new(excelize.DefaultGridColor),
+	new(excelize.RightToLeft),
+	new(excelize.ShowFormulas),
+	new(excelize.ShowGridLines),
+	new(excelize.ShowRowColHeaders),
 }
 
 var _ = []excelize.SheetViewOptionPtr{