screen update

example/main.go

@@ -1,62 +1,62 @@
 package main
 
 import (
-    "github.com/orus-dev/osui"
-    "github.com/orus-dev/osui/colors"
-    "github.com/orus-dev/osui/ui"
+	"github.com/orus-dev/osui"
+	"github.com/orus-dev/osui/colors"
+	"github.com/orus-dev/osui/ui"
 )
 
 func main() {
-    screen := osui.NewScreen(App())
-    screen.Run()
+	screen := osui.NewScreen(App())
+	screen.Run()
 }
 
 func App() *ui.PaginatorComponent {
-    w, h := osui.GetTerminalSize()
+	w, h := osui.GetTerminalSize()
 
-    var keybindings ui.Class[*ui.TextComponent]
-    keybindings.SetProperties(func(tc *ui.TextComponent) *ui.TextComponent {
-        tc.Text = colors.Blue + tc.Text
-        tc.Data.Y = h - 2
-        return tc
-    })
+	var keybindings ui.Class[*ui.TextComponent]
+	keybindings.SetProperties(func(tc *ui.TextComponent) *ui.TextComponent {
+		tc.Text = colors.Blue + tc.Text
+		tc.Data.Y = h - 3
+		return tc
+	})
 
-    return ui.Paginator(
-        // Page 1
-        ui.Div(
-            ui.Text("Welcome to the example! The blue text on the bottom show the keybindings"),
-            keybindings.Class(ui.Text("Next Page: Tab. Previous Page: shift + tab")),
-        ),
+	return ui.Paginator(
+		// Page 1
+		ui.Div(
+			ui.Text("Welcome to the example! The blue text on the bottom show the keybindings"),
+			keybindings.Class(ui.Text("Next Page: Tab. Previous Page: shift + tab")),
+		),
 
-        // Page 2
-        ui.Div(
-            ui.WithPosition(0, 0, ui.Text("This blue square is a div. A div stores multiple components into one")),
-            ui.WithPosition((w-30)/2, (h-4)/2,
-                ui.WithSize(30, 5,
-                    ui.Div(ui.WithPosition(4, 1, ui.Button("Hello, World!").Params(ui.ButtonParams{Width: 19}))).Params(ui.DivParams{Style: ui.DivStyle{Outline: colors.Blue}}),
-                )),
-            keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D)")),
-        ),
+		// Page 2
+		ui.Div(
+			ui.WithPosition(0, 0, ui.Text("This blue square is a div. A div stores multiple components into one")),
+			ui.WithPosition((w-30)/2, (h-4)/2,
+				ui.WithSize(32, 7,
+					ui.Div(ui.WithPosition(2, 2, ui.Text("This text is inside a div!"))).Params(ui.DivParams{Style: ui.DivStyle{Outline: colors.Blue, Foreground: colors.Red}}),
+				)),
+			keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D)")),
+		),
 
-        // Page 3
-        ui.Div(
-            ui.WithPosition(0, 0, ui.Text("This is a InputBox. It takes user input just like a GUI. "+colors.Red+colors.Bold+"NOTE: This InputBox is inside a div")),
-            ui.WithPosition((w-32)/2, (h-3)/2, ui.InputBox(30)),
-            keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Finish: Enter.")),
-        ),
+		// Page 3
+		ui.Div(
+			ui.WithPosition(0, 0, ui.Text("This is a InputBox. It takes user input just like a GUI. "+colors.Red+colors.Bold+"NOTE: This InputBox is inside a div")),
+			ui.WithPosition((w-32)/2, (h-3)/2, ui.InputBox(30)),
+			keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Finish: Enter.")),
+		),
 
-        // Page 4
-        ui.Div(
-            ui.WithPosition(0, 0, ui.Text("This is a Button. To click it press Enter. "+colors.Red+colors.Bold+"NOTE: This Button is inside a div")),
-            ui.WithPosition((w-20)/2, (h-3)/2, ui.Button("This is a button")),
-            keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Click Button: Enter")),
-        ),
+		// Page 4
+		ui.Div(
+			ui.WithPosition(0, 0, ui.Text("This is a Button. To click it press Enter. "+colors.Red+colors.Bold+"NOTE: This Button is inside a div")),
+			ui.WithPosition((w-20)/2, (h-3)/2, ui.Button("This is a button")),
+			keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Click Button: Enter")),
+		),
 
-        // Page 5
-        ui.Div(
-            ui.WithPosition(0, 0, ui.Text("This is a Menu. It prompts you to select an item. "+colors.Red+colors.Bold+"NOTE: This Menu is inside a div")),
-            ui.WithPosition(0, 2, ui.Menu("Item 1", "Item 2", "Item 3", "Item 4", "Item 5").Params(ui.MenuParams{OnSelected: func(mc *ui.MenuComponent, b bool) {}})),
-            keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Navigate Menu: W/S. Select: Enter. Exit Menu: Q. ")),
-        ),
-    )
+		// Page 5
+		ui.Div(
+			ui.WithPosition(0, 0, ui.Text("This is a Menu. It prompts you to select an item. "+colors.Red+colors.Bold+"NOTE: This Menu is inside a div")),
+			ui.WithPosition(0, 2, ui.Menu("Item 1", "Item 2", "Item 3", "Item 4", "Item 5").Params(ui.MenuParams{OnSelected: func(mc *ui.MenuComponent, b bool) {}})),
+			keybindings.Class(ui.Text("Navigate: Ctrl+(W/A/S/D). Navigate Menu: W/S. Select: Enter. Exit Menu: Q. ")),
+		),
+	)
 }

osui.go

@@ -2,9 +2,11 @@ package osui
 
 import (
 	"fmt"
+	"os"
 	"strings"
 
 	"github.com/orus-dev/osui/colors"
+	"golang.org/x/term"
 )
 
 type ComponentData struct {
@@ -24,19 +26,19 @@ type Component interface {
 }
 
 type Screen struct {
-	component Component
+	Component Component
 }
 
 func NewScreen(c Component) *Screen {
 	HideCursor()
-	s := &Screen{component: c}
+	s := &Screen{Component: c}
 	return s
 }
 
 func (s *Screen) Render() {
 	width, height := GetTerminalSize()
 	frame := NewFrame(width, height)
-	data := s.component.GetComponentData()
+	data := s.Component.GetComponentData()
 	if data.Height == 0 {
 		data.Height = height
 	}
@@ -46,18 +48,23 @@ func (s *Screen) Render() {
 	data.Screen = s
 	data.IsActive = true
 	data.DefaultColor = colors.Reset
-	RenderOnFrame(s.component, &frame)
+	RenderOnFrame(s.Component, &frame)
 	Clear()
-	fmt.Print(strings.Join(frame, "\n"))
+	fmt.Print(strings.Join(frame, ""))
 }
 
 func (s *Screen) Run() {
-	data := s.component.GetComponentData()
+	oldState, err := term.MakeRaw(int(os.Stdin.Fd()))
+	if err != nil {
+		panic(err)
+	}
+	defer term.Restore(int(os.Stdin.Fd()), oldState)
+	data := s.Component.GetComponentData()
 	data.Screen = s
 	for {
 		s.Render()
 		k, _ := ReadKey()
-		if s.component.Update(k) {
+		if s.Component.Update(k) {
 			ShowCursor()
 			return
 		}

ui/button.go

@@ -45,30 +45,30 @@ func (b *ButtonComponent) Render() string {
 
 	if b.Clicked {
 		return fmt.Sprintf(" %s\n%s│%s│%s\n %s",
-			colors.Reset+b.Style.ClickedOutline+strings.Repeat("_", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+			colors.Reset+b.Style.ClickedOutline+strings.Repeat("_", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 			colors.Reset+b.Style.ClickedOutline,
-			colors.Reset+colors.Combine(b.Style.ClickedBackground, b.Style.ClickedForeground)+centerText(b.Text, b.Data.Width)+colors.Reset+b.Style.ClickedOutline,
+			colors.Reset+colors.Combine(b.Style.ClickedBackground, b.Style.ClickedForeground)+centerText(b.Text, b.Data.Width-2)+colors.Reset+b.Style.ClickedOutline,
 			colors.Reset+b.Data.DefaultColor,
-			colors.Reset+b.Style.ClickedOutline+strings.Repeat("‾", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+			colors.Reset+b.Style.ClickedOutline+strings.Repeat("‾", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 		)
 	}
 
 	if b.Data.IsActive {
 		return fmt.Sprintf(" %s\n%s│%s│%s\n %s",
-			colors.Reset+b.Style.ActiveOutline+strings.Repeat("_", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+			colors.Reset+b.Style.ActiveOutline+strings.Repeat("_", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 			colors.Reset+b.Style.ActiveOutline,
-			colors.Reset+colors.Combine(b.Style.ActiveBackground, b.Style.ActiveForeground)+centerText(b.Text, b.Data.Width)+colors.Reset+b.Style.ActiveOutline,
+			colors.Reset+colors.Combine(b.Style.ActiveBackground, b.Style.ActiveForeground)+centerText(b.Text, b.Data.Width-2)+colors.Reset+b.Style.ActiveOutline,
 			colors.Reset+b.Data.DefaultColor,
-			colors.Reset+b.Style.ActiveOutline+strings.Repeat("‾", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+			colors.Reset+b.Style.ActiveOutline+strings.Repeat("‾", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 		)
 	}
 
 	return fmt.Sprintf(" %s\n%s│%s│%s\n %s",
-		colors.Reset+b.Style.Outline+strings.Repeat("_", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+		colors.Reset+b.Style.Outline+strings.Repeat("_", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 		colors.Reset+b.Style.Outline,
-		colors.Reset+colors.Combine(b.Style.Background, b.Style.Foreground)+centerText(b.Text, b.Data.Width)+colors.Reset+b.Style.Outline,
+		colors.Reset+colors.Combine(b.Style.Background, b.Style.Foreground)+centerText(b.Text, b.Data.Width-2)+colors.Reset+b.Style.Outline,
 		colors.Reset+b.Data.DefaultColor,
-		colors.Reset+b.Style.Outline+strings.Repeat("‾", b.Data.Width)+colors.Reset+b.Data.DefaultColor,
+		colors.Reset+b.Style.Outline+strings.Repeat("‾", b.Data.Width-2)+colors.Reset+b.Data.DefaultColor,
 	)
 }
 

ui/div.go

@@ -33,7 +33,7 @@ func (d *DivComponent) GetComponentData() *osui.ComponentData {
 
 func (d *DivComponent) Render() string {
 	osui.UseStyle(d.Style)
-	frame := osui.NewFrame(d.Data.Width, d.Data.Height)
+	frame := osui.NewFrame(d.Data.Width-2, d.Data.Height-2)
 	for i, c := range d.Components {
 		data := c.GetComponentData()
 		if i == d.ActiveComponent {
@@ -42,10 +42,10 @@ func (d *DivComponent) Render() string {
 			data.IsActive = false
 		}
 		if data.Width == 0 {
-			data.Width = d.Data.Width
+			data.Width = d.Data.Width - 2
 		}
 		if data.Height == 0 {
-			data.Height = d.Data.Height
+			data.Height = d.Data.Height - 2
 		}
 		data.DefaultColor = colors.Combine(d.Style.Background, d.Style.Foreground)
 		data.Screen = d.Data.Screen
@@ -58,10 +58,10 @@ func (d *DivComponent) Render() string {
 		return strings.Join(frame, "\n")
 	} else {
 		for i, f := range frame {
-			frame[i] = d.Style.Outline + "│" + colors.Combine(d.Style.Foreground, d.Style.Background) + f + colors.Reset + d.Style.Outline + "│" + colors.Reset
+			frame[i] = d.Style.Outline + "│" + colors.Reset + colors.Combine(d.Style.Foreground, d.Style.Background) + f + colors.Reset + d.Style.Outline + "│" + colors.Reset
 		}
 	}
-	return " " + d.Style.Outline + strings.Repeat("_", d.Data.Width) + colors.Reset + "\n" + strings.Join(frame, "\n") + "\n " + d.Style.Outline + strings.Repeat("‾", d.Data.Width) + colors.Reset
+	return " " + d.Style.Outline + strings.Repeat("_", d.Data.Width-2) + colors.Reset + "\n" + strings.Join(frame, "\n") + "\n " + d.Style.Outline + strings.Repeat("‾", d.Data.Width-2) + colors.Reset
 }
 
 func (d *DivComponent) Update(key string) bool {

utils.go

@@ -11,16 +11,18 @@ import (
 
 	"github.com/nathan-fiscaletti/consolesize-go"
 	"github.com/orus-dev/osui/colors"
-	"golang.org/x/term"
 )
 
-func RenderLine(frame_, line_ string, x int, fm map[int]string, lm map[int]string) string {
+var re = regexp.MustCompile(`(\x1b\[([0-9;]*)[a-zA-Z])+`)
+
+func RenderLine(frame_, line_ string, x int, fm, lm map[int]string) string {
 	var res strings.Builder
 	frame := []rune(frame_)
 	line := []rune(line_)
 
-	for i := range frame {
-		if i >= x && i-x < len(line) {
+	flen, llen := len(frame), len(line)
+	for i := 0; i < flen; i++ {
+		if i >= x && i-x < llen {
 			if v, ok := lm[i-x]; ok {
 				res.WriteString(v)
 			}
@@ -36,8 +38,6 @@ func RenderLine(frame_, line_ string, x int, fm map[int]string, lm map[int]strin
 }
 
 func CompressString(input string) (string, map[int]string) {
-	pattern := `(\x1b\[([0-9;]*)[a-zA-Z])+`
-	re := regexp.MustCompile(pattern)
 	matchesMap := make(map[int]string)
 	res := []rune{}
 
@@ -70,11 +70,6 @@ func RenderOnFrame(c Component, frame *[]string) {
 }
 
 func ReadKey() (string, error) {
-	oldState, err := term.MakeRaw(int(os.Stdin.Fd()))
-	if err != nil {
-		return "", err
-	}
-	defer term.Restore(int(os.Stdin.Fd()), oldState)
 	var b [3]byte
 	n, err := os.Stdin.Read(b[:])
 	if err != nil {