Skip to content

Update deepdives to capture new Scrolling API#4748

Merged
tig merged 4 commits intov2_developfrom
copilot/update-deepdives-scrolling-api
Feb 24, 2026
Merged

Update deepdives to capture new Scrolling API#4748
tig merged 4 commits intov2_developfrom
copilot/update-deepdives-scrolling-api

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 24, 2026

Proposed Changes/Todos

Updates scrolling.md and layout.md to document the new Scrolling API that replaced ScrollBar.AutoShow with ScrollBarVisibilityMode enum and ViewportSettingsFlags.

  • Document ScrollBarVisibilityMode enum (Manual, Auto, Always, None)
  • Document ViewportSettingsFlags.HasVerticalScrollBar and HasHorizontalScrollBar
  • Expand ViewportSettings section with all flag categories
  • Replace all AutoShow references with new API patterns
  • Update code examples in View.md, migratingfromv1.md, newinv2.md

New API Pattern:

// Old (deprecated)
view.VerticalScrollBar.AutoShow = true;

// New (recommended)
view.ViewportSettings |= ViewportSettingsFlags.HasVerticalScrollBar;

// Alternative (manual control)
view.VerticalScrollBar.VisibilityMode = ScrollBarVisibilityMode.Always;

Updated Documentation:

  • scrolling.md - Added ScrollBarVisibilityMode section, expanded ViewportSettings with ScrollBar flags
  • layout.md - Added ScrollBar flags to ViewportSettings section
  • View.md, migratingfromv1.md, newinv2.md - Updated all code examples

Pull Request checklist:

  • I've named my PR in the form of "Fixes #issue. Terse description."
  • My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
  • My code follows the Terminal.Gui library design guidelines
  • I ran dotnet test before commit
  • I have made corresponding changes to the API documentation (using /// style comments)
  • My changes generate no new warnings
  • I have checked my code and corrected any poor grammar or misspellings
  • I conducted basic QA to assure all features are working
Original prompt

This section details on the original issue you should resolve

<issue_title>Deepdives do not capture new Scrolling API</issue_title>
<issue_description>These deep dives need updating based on the merged #4714

  • scrolling.md
  • layout.md

</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits February 24, 2026 08:32
Co-authored-by: tig <585482+tig@users.noreply.github.com>
…w with new API

Co-authored-by: tig <585482+tig@users.noreply.github.com>
Co-authored-by: tig <585482+tig@users.noreply.github.com>
Copilot AI changed the title [WIP] Update deep dives to capture new Scrolling API Update deepdives to capture new Scrolling API Feb 24, 2026
Copilot AI requested a review from tig February 24, 2026 08:36
@tig tig marked this pull request as ready for review February 24, 2026 14:37
@tig tig merged commit e04b004 into v2_develop Feb 24, 2026
1 check passed
@tig tig deleted the copilot/update-deepdives-scrolling-api branch February 24, 2026 14:37
tig added a commit to tig/Terminal.Gui that referenced this pull request Feb 25, 2026
Resolved 5 conflicts:
- ApplicationImpl.Driver.cs: kept both using statements
- ApplicationImpl.Lifecycle.cs: took v2_develop guard clause fix
- MainLoopCoordinator.cs: took v2_develop guard clause refactor
- ColorPicker.16.cs: kept HEAD's OnActivated override (new CWP pattern)
- View.md: took v2_develop's updated scrolling docs from PR gui-cs#4748

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

Deepdives do not capture new Scrolling API

2 participants