Merge pull request #19 from pdeffebach/newwindow

Add `newwindow` keyword argument

Author: pdeffebach

src/FloatingTableView.jl

@@ -5,7 +5,7 @@ using TableView
 
 export browse, showtable
 
-# Method for keeping current window active borrowed 
+# Method for keeping current window active borrowed
 # from Plots.jl
 mutable struct CurrentWin
     nullablewin::Union{Window, Nothing}
@@ -28,18 +28,21 @@ end
 current(win::Window) = (CURRENT_WIN.nullablewin = win)
 
 """
-	browse(t; kwargs)
+	browse(t; newwindow = false, kwargs...)
 
-Browse data source in a new Blink window using 
-Tables.jl's `showtable` function. 
+Browse data source in a new Blink window using
+Tables.jl's `showtable` function.
 
 # Arguments
 
-- `t`, a Tables.jl-compatible data source, 
-  for example a `DataFrame` or a `NamedTuple` 
+- `t`, a Tables.jl-compatible data source,
+  for example a `DataFrame` or a `NamedTuple`
   of `Vector`s
-- `kwargs`, keyword arguments passed to the 
-  `showtable` command from TableView.jl to 
+- `newwindow`, boolean keyword argument to view the data
+  in a new `Blink` window instead of over-writing.
+  Viewing in a new window will be slower than over-writing.
+- `kwargs...`, keyword arguments passed to the
+  `showtable` command from TableView.jl to
   control the details of the new data viewer.
   See [`showtable`](@ref) for details.
 
@@ -59,8 +62,10 @@ julia> browse(t, dark = true)
 
 See also: [`showtable`](@ref)
 """
-function browse(df; height = "100vh", kwargs...)
-	if iswinnull() || !active(current())
+function browse(df; height = "100vh", newwindow::Bool = false, kwargs...)
+	if newwindow == true
+    body!(Window(), showtable(df; height = height, kwargs...))
+  elseif iswinnull() || !active(current())
 		w = Window()
 		current(w)
 		body!(current(), showtable(df; height = height, kwargs...))

test/runtests.jl

@@ -22,6 +22,9 @@ const W = FloatingTableView.TableView.WebIO
 	t = [1 3; 3 4]
 	@test showtable(t) isa W.Scope
     @test browse(t) === nothing
+
+    t = [(a = 1, b = 2), (a = 3, b = 4)]
+    @test browse(t; newwindow=true) === nothing
 end
 
 end