Update Window.SetIcon

go.mod

@@ -3,7 +3,7 @@ module git.tebibyte.media/tomo/backend
 go 1.20
 
 require (
-	git.tebibyte.media/tomo/tomo v0.35.0
+	git.tebibyte.media/tomo/tomo v0.36.0
 	git.tebibyte.media/tomo/typeset v0.7.1
 	git.tebibyte.media/tomo/xgbkb v1.0.1
 	github.com/jezek/xgb v1.1.1

go.sum

@@ -1,6 +1,6 @@
 git.tebibyte.media/sashakoshka/xgbkb v1.0.0/go.mod h1:pNcE6TRO93vHd6q42SdwLSTTj25L0Yzggz7yLe0JV6Q=
-git.tebibyte.media/tomo/tomo v0.35.0 h1:1XvcUcWg1rBZXov3KfuX6VfiuBQ2mcJHIslHMLn07no=
-git.tebibyte.media/tomo/tomo v0.35.0/go.mod h1:C9EzepS9wjkTJjnZaPBh22YvVPyA4hbBAJVU20Rdmps=
+git.tebibyte.media/tomo/tomo v0.36.0 h1:V9vyPYb4kpUceBhcDF/XyLDACzE5lY8kYEGHAkIsqs0=
+git.tebibyte.media/tomo/tomo v0.36.0/go.mod h1:C9EzepS9wjkTJjnZaPBh22YvVPyA4hbBAJVU20Rdmps=
 git.tebibyte.media/tomo/typeset v0.7.1 h1:aZrsHwCG5ZB4f5CruRFsxLv5ezJUCFUFsQJJso2sXQ8=
 git.tebibyte.media/tomo/typeset v0.7.1/go.mod h1:PwDpSdBF3l/EzoIsa2ME7QffVVajnTHZN6l3MHEGe1g=
 git.tebibyte.media/tomo/xgbkb v1.0.1 h1:b3HDUopjdQp1MZrb5Vpil4bOtk3NnNXtfQW27Blw2kE=

x/event.go

@@ -13,7 +13,7 @@ type scrollSum struct {
 }
 
 // TODO: this needs to be configurable, we need a config api
-const scrollDistance = 16
+const scrollDistance = 32
 
 func (sum *scrollSum) add (button xproto.Button, window *window, state uint16) {
 	if xgbkb.StateToModifiers(state).Shift {

x/window.go

@@ -18,7 +18,6 @@ import "github.com/jezek/xgbutil/keybind"
 import "github.com/jezek/xgbutil/mousebind"
 import "github.com/jezek/xgbutil/xgraphics"
 
-type mainWindow struct { *window }
 type window struct {
 	backend   *Backend
 	hierarchy *system.Hierarchy
@@ -28,6 +27,7 @@ type window struct {
 
 	title string
 
+	leader      *window
 	modalParent *window
 	hasModal    bool
 	shy         bool
@@ -63,28 +63,24 @@ func (this *windowLink) NotifyMinimumSizeChange () {
 func (this *Backend) NewWindow (
 	bounds image.Rectangle,
 ) (
-	output tomo.MainWindow,
+	output tomo.Window,
 	err error,
 ) {
 	this.assert()
-	window, err := this.newWindow(bounds, false)
-	
-	output = mainWindow { window: window }
-	return output, err
+	return this.newWindow(bounds, false)
 }
 
 func (this *Backend) NewPlainWindow (
 	bounds image.Rectangle,
 ) (
-	output tomo.MainWindow,
+	output tomo.Window,
 	err error,
 ) {
 	this.assert()
 	window, err := this.newWindow(bounds, false)
 	window.setType("dock")
 	
-	output = mainWindow { window: window }
-	return output, err
+	return window, err
 }
 
 func (this *Backend) newWindow (
@@ -100,6 +96,7 @@ func (this *Backend) newWindow (
 	window := &window     { backend: this   }
 	link   := &windowLink { window:  window }
 	window.hierarchy = this.system.NewHierarchy(link)
+	window.leader    = window
 
 	window.xWindow, err = xwindow.Generate(this.x)
 	if err != nil { return }
@@ -178,10 +175,11 @@ func (this *window) SetTitle (title string) {
 	icccm.WmIconNameSet (this.backend.x, this.xWindow.Id, title)
 }
 
-func (this *window) SetIcon (sizes ...image.Image) {
+func (this *window) SetIcon (sizes ...canvas.Texture) {
 	wmIcons := []ewmh.WmIcon { }
 
 	for _, icon := range sizes {
+		// TODO we use textures now. make this better
 		width  := icon.Bounds().Max.X
 		height := icon.Bounds().Max.Y
 		wmIcon := ewmh.WmIcon {
@@ -247,18 +245,23 @@ func (this *window) NewModal (bounds image.Rectangle) (tomo.Window, error) {
 	return modal, err
 }
 
-func (this mainWindow) NewChild (bounds image.Rectangle) (tomo.Window, error) {
+func (this *window) NewChild (bounds image.Rectangle) (tomo.Window, error) {
+	leader := this.leader
+	
 	child, err := this.backend.newWindow (
 		bounds.Add(this.metrics.bounds.Min), false)
+	child.leader = leader
 	if err != nil { return nil, err }
-	child.setClientLeader(this.window)
-	this.setClientLeader(this.window)
+	
+	child.setClientLeader(leader)
+	leader.setClientLeader(leader)
+	
 	icccm.WmTransientForSet (
 		this.backend.x,
-		this.xWindow.Id,
-		this.xWindow.Id)
-	this.setType("UTILITY")
-	// this.inheritProperties(this.window)
+		child.xWindow.Id,
+		leader.xWindow.Id)
+	child.setType("UTILITY")
+	// child.inheritProperties(leader.window)
 	return this, err
 }