[GH-ISSUE #877] border color doesn't seem to work for Modal primitive #639

New issue

Closed

opened 2026-03-04 01:06:40 +03:00 by kerem · 1 comment

kerem commented

2026-03-04 01:06:40 +03:00

Owner

Originally created by @justmeandopensource on GitHub (Aug 31, 2023).
Original GitHub issue: https://github.com/rivo/tview/issues/877

Hi,
First of all thanks for this wonderful library.
I am trying to show a Modal dialog with the specified border color.

modal := tview.NewModal()
modal.SetText("Delete the transaction?")
modal.AddButtons([]string{"Yes", "No"})
modal.SetBackgroundColor(tcell.Color235)
modal.SetBorder(true)
modal.SetBorderColor(tview.Styles.SecondaryTextColor)

But the border is always white.
It works for other primitives though.

table := tview.Table()
table.SetBorderColor(tview.Styles.SecondaryTextColor)

It could be me not understanding the library better. Could someone please share some light on this?
Thanks

Originally created by @justmeandopensource on GitHub (Aug 31, 2023). Original GitHub issue: https://github.com/rivo/tview/issues/877 Hi, First of all thanks for this wonderful library. I am trying to show a Modal dialog with the specified border color. ``` modal := tview.NewModal() modal.SetText("Delete the transaction?") modal.AddButtons([]string{"Yes", "No"}) modal.SetBackgroundColor(tcell.Color235) modal.SetBorder(true) modal.SetBorderColor(tview.Styles.SecondaryTextColor) ``` But the border is always white. It works for other primitives though. ``` table := tview.Table() table.SetBorderColor(tview.Styles.SecondaryTextColor) ``` It could be me not understanding the library better. Could someone please share some light on this? Thanks

kerem closed this issue

2026-03-04 01:06:40 +03:00

kerem commented

2026-03-04 01:06:41 +03:00

Author

Owner

@justmeandopensource commented on GitHub (Sep 1, 2023):

@digitallyserviced Thanks for the clarification. It now makes sense.

@justmeandopensource commented on GitHub (Sep 1, 2023): @digitallyserviced Thanks for the clarification. It now makes sense.