Skip to content
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 NewTabWidgetWithStyle() #402

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dayvonjersen
Copy link

tabWidget, err := walk.NewTabWidgetWithStyle(mw, win.TCS_MULTILINE)
checkErr(err)
mw.Children().Add(tabWidget)

multilinetabs

 - enables a user to apply these styles to a tab widget:
   https://docs.microsoft.com/en-us/windows/desktop/Controls/tab-control-styles

 - follows the convention of other functions in this package
   like NewCompositeWithStyle and NewListBoxWithStyle
@zx2c4
Copy link
Contributor

zx2c4 commented May 26, 2019

Are there other useful styles? Why not instead expose this as SetMultiline(true)?

dayvonjersen added a commit to dayvonjersen/chopsuey that referenced this pull request Sep 3, 2023
 - gofmt decided my comments weren't formatted properly I guess
 - replace all calls to win.MustLoadLibrary() and
   win.MustGetProcAddress()
   thanks to lxn/win@e90c084

    annoying af tbh

 - my PR for NewTabWidgetWithStyle lxn/walk#402
   never got merged in any way and since go modules made $GOPATH
   useless, there's no easy way to get multiline tabs right now so I
   just disabled them. I'll fork the repo later or something.

 - added a nil pointer check which fixes one (1) of the crashes

 - some widgets now have ~20px of padding which is annoying

 - this whole codebase is a superfund site tbh might be better off
   rewriting the whole thing from scratch including writing my own IRC
   lib, I suspect goirc's async nature is what has caused all the
   complexity ""requiring"" tabManager and mw.Synchronize calls and so
   forth. It's also possible the multiline tabs are buggy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants