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

FIX(container): add menu containers public re-export #14

Closed
wants to merge 1 commit into from

Conversation

liketechnik
Copy link

@liketechnik liketechnik commented Aug 4, 2020

Really like the concept of this library and was eager to try some stuff out.

Was really happy to find the menu containers in the documentation, but the compiler complained they were private in the module.

I'm not totally sure if my fix is the correct/planned way for adding the menu containers, but it seems to work :D

(P.S.: Sorry for the push spam. Apparently I should really start looking at the details of my work before pushing, because I'm too stupid to get them right the first time...)

@liketechnik liketechnik force-pushed the fix-menu_visibility branch from b2ec4f0 to 1707d74 Compare August 4, 2020 16:52
@liketechnik liketechnik force-pushed the fix-menu_visibility branch from 1707d74 to e5ebc65 Compare August 4, 2020 16:58
@emilk
Copy link
Owner

emilk commented Aug 5, 2020

Menus are barely half-done, but feel free to give them a spin!

I've exposed them now in 56352c2

Use it as:

use egui::menu;

menu::bar(ui, |ui| {
    menu::menu(ui, "File", |ui| {
        if ui.add(Button::new("Open")).clicked {
            ...
        }
    });
});

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