Skip to content

Remove fullscreen button from MindElixir toolbar#18

Open
DisciplinedSoftware wants to merge 2 commits intoOlegIGalkin:mainfrom
DisciplinedSoftware:fix/remove-fullscreen-host-side
Open

Remove fullscreen button from MindElixir toolbar#18
DisciplinedSoftware wants to merge 2 commits intoOlegIGalkin:mainfrom
DisciplinedSoftware:fix/remove-fullscreen-host-side

Conversation

@DisciplinedSoftware
Copy link
Copy Markdown

Summary

  • removes the fullscreen toolbar button from the embedded MindElixir toolbar
  • keeps host-side control of fullscreen behavior instead of exposing it in the in-map toolbar
  • updates both bundled copies used by VS Code and Visual Studio variants

Commits

  1. Revert fullscreen button fix in MindElixir.js
  2. Remove fullscreen button

Notes

  • source branch: DisciplinedSoftware:fix/remove-fullscreen-host-side
  • target branch: main

Copilot AI review requested due to automatic review settings March 24, 2026 16:55
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to remove the fullscreen control from the embedded MindElixir toolbar and keep fullscreen behavior controlled by the host (VS Code / Visual Studio).

Changes:

  • Adds CSS overrides in both hosts to hide the #fullscreen toolbar item and adjust toolbar spacing.
  • Updates both bundled MindElixir.js copies, but currently (re-)introduces a fullscreen toolbar button and fullscreen handling code in the library bundle.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
Visual Studio/CodeMindMap/MindElixir/MindElixir.js Modifies toolbar construction and adds fullscreen handling logic.
Visual Studio/CodeMindMap/CodeMindMapHtml.cs Adds host CSS to hide the fullscreen toolbar item and tweak layout.
VS Code/src/extension.ts Adds host CSS to hide the fullscreen toolbar item and tweak layout.
VS Code/MindElixir/MindElixir.js Same toolbar + fullscreen handling changes as the Visual Studio bundle.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Visual Studio/CodeMindMap/MindElixir/MindElixir.js
Comment thread VS Code/MindElixir/MindElixir.js
Comment thread Visual Studio/CodeMindMap/MindElixir/MindElixir.js
Comment thread Visual Studio/CodeMindMap/CodeMindMapHtml.cs
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