Skip to content

Hide output from visible text editors#2918

Closed
AndreasArvidsson wants to merge 2 commits into
mainfrom
output
Closed

Hide output from visible text editors#2918
AndreasArvidsson wants to merge 2 commits into
mainfrom
output

Conversation

@AndreasArvidsson

Copy link
Copy Markdown
Member

Today we waste hats on the output which shouldn't really be a text editor

@AndreasArvidsson AndreasArvidsson requested a review from a team as a code owner May 21, 2025 06:11
@pokey

pokey commented May 21, 2025

Copy link
Copy Markdown
Member

Do you have a screenshot of where these hats are?

@AndreasArvidsson

Copy link
Copy Markdown
Member Author

Output panel
image

@pokey

pokey commented May 22, 2025

Copy link
Copy Markdown
Member

I actually consider that a feature. They only show up if the panel is visible right?

@AndreasArvidsson

Copy link
Copy Markdown
Member Author

I wasn't sure if I liked it or not. You can't edit in the output, but you can bring from it of course.

Yes

@AndreasArvidsson

Copy link
Copy Markdown
Member Author

If you think we should keep this functionality it's fine with me?

@pokey

pokey commented May 29, 2025

Copy link
Copy Markdown
Member

I'd be inclined to keep today's functionality. If we can put hats on visible things, I think we should. The active editor gets preferred hats anyway I assume so seems like there's not much lost

@AndreasArvidsson

Copy link
Copy Markdown
Member Author

Agreed

@AndreasArvidsson AndreasArvidsson deleted the output branch July 12, 2025 12:24
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