Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add with_taskbar to viewport builder #3958

Merged
merged 5 commits into from
Feb 6, 2024

Conversation

AnotherNathan
Copy link
Contributor

Allows removing the taskbar icon of a viewport.

This can be useful when making an overlay type viewport (window is always on top).

@@ -442,6 +445,13 @@ impl ViewportBuilder {
self
}

/// windows: Whether show or hide the window icon in the taskbar.
#[inline]
pub fn with_skip_taskbar(mut self, skip: bool) -> Self {
Copy link
Owner

@emilk emilk Feb 5, 2024

Choose a reason for hiding this comment

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

Please invert this to with_taskbar to avoid double negation (with_skip_taskbar(false))

@AnotherNathan AnotherNathan changed the base branch from master to emilk/dock February 5, 2024 16:13
@AnotherNathan AnotherNathan changed the base branch from emilk/dock to master February 5, 2024 16:13
@AnotherNathan AnotherNathan changed the title add with_skip_taskbar to viewport builder add with_taskbar to viewport builder Feb 5, 2024
@AnotherNathan AnotherNathan requested a review from emilk February 5, 2024 16:22
@emilk emilk added eframe Relates to epi and eframe egui labels Feb 6, 2024
@emilk emilk merged commit ee7fb47 into emilk:master Feb 6, 2024
20 of 21 checks passed
hacknus pushed a commit to hacknus/egui that referenced this pull request Oct 30, 2024
Allows removing the taskbar icon of a viewport.

This can be useful when making an overlay type viewport (window is
always on top).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eframe Relates to epi and eframe egui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants