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

Set Rerun viewer native app icon using eframe #1976

Merged
merged 2 commits into from Apr 26, 2023
Merged

Conversation

emilk
Copy link
Member

@emilk emilk commented Apr 26, 2023

What

This code was recently ported to eframe,

Test:

  • Mac
  • Windows
  • X11

Checklist

PR Build Summary: https://build.rerun.io/pr/1976

This was recently added to eframe
@emilk emilk added 🕸️ web regarding running the viewer in a browser 🪟 windows Problems specific to the Windows OS 📺 re_viewer affects re_viewer itself and removed 🕸️ web regarding running the viewer in a browser labels Apr 26, 2023
@jleibs
Copy link
Member

jleibs commented Apr 26, 2023

X11 working:

image

@jleibs
Copy link
Member

jleibs commented Apr 26, 2023

On windows the alt-tab icon and menu-bar icon are correct but the task-bar icon is still the python shell icon from the launching context:
image

This is the same behavior as previously though, so not a regression.

@emilk
Copy link
Member Author

emilk commented Apr 26, 2023

niiice that X11 is working! :D

@emilk emilk merged commit 57964f6 into main Apr 26, 2023
15 checks passed
@emilk emilk deleted the emilk/eframe-app-icon branch April 26, 2023 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📺 re_viewer affects re_viewer itself 🪟 windows Problems specific to the Windows OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants