[PR #610] [CLOSED] Add support for styling the borders used with modals. #994

Closed
opened 2026-03-04 01:08:55 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/rivo/tview/pull/610
Author: @gdamore
Created: 5/28/2021
Status: Closed

Base: masterHead: modal-borders


📝 Commits (1)

  • 4f66010 Add support for styling the borders used with modals.

📊 Changes

1 file changed (+12 additions, -0 deletions)

View changed files

📝 modal.go (+12 -0)

📄 Description

It turns out that I need to have more control over the styling of these borders. This small addition fixes that. (Note that due to inheritance from Box we had this method already, but it basically didn't do what you expected -- we need to control the frame's borders, not the box's.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/rivo/tview/pull/610 **Author:** [@gdamore](https://github.com/gdamore) **Created:** 5/28/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `modal-borders` --- ### 📝 Commits (1) - [`4f66010`](https://github.com/rivo/tview/commit/4f66010c494174e03a7558c5ee54b01f4434accd) Add support for styling the borders used with modals. ### 📊 Changes **1 file changed** (+12 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `modal.go` (+12 -0) </details> ### 📄 Description It turns out that I need to have more control over the styling of these borders. This small addition fixes that. (Note that due to inheritance from Box we had this method already, but it basically didn't do what you expected -- we need to control the frame's borders, not the box's. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-04 01:08:55 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/tview#994
No description provided.