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.

Sign up for GitHub

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

Jump to bottom

Problem with setTheme method for Overlay components. #5378

Closed
vaadin-bot opened this issue Jul 2, 2014 · 3 comments
Closed

Problem with setTheme method for Overlay components. #5378

vaadin-bot opened this issue Jul 2, 2014 · 3 comments
Labels
bug
Milestone
Vaadin 7.3.0.rc1

Comments

@vaadin-bot
Copy link
Collaborator

Originally by Dmitrii Rogozin

Calling setTheme method #2874 breaks popup window. Probably breaks some other overlay component.

See test UI example

Imported from https://dev.vaadin.com/ issue #14139
@vaadin-bot
Copy link
Collaborator Author

Originally by Dmitrii Rogozin

Attachment added: ThemeChangeOnTheFlyForPopup.java (1.7 KiB)

@vaadin-bot
Copy link
Collaborator Author

Originally by Dmitrii Rogozin

Probably connected with #14124.

A place to start may be VOverlay.getOverlayContainer(ApplicationContext ac) method (line 854). GetStyleName method returns empty string, while it should return a theme.

@vaadin-bot
Copy link
Collaborator Author

Originally by @Artur-

Added updating of the overlay container class name in addition to the UI element class name

@vaadin-bot vaadin-bot added the bug label Dec 10, 2016
@vaadin-bot vaadin-bot added this to the Vaadin 7.3.0.rc1 milestone Dec 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Vaadin 7.3.0.rc1
Development

No branches or pull requests
1 participant
@vaadin-bot