Skip to content

Splashscreen fixes #851

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
Closed
wants to merge 2 commits into from
Closed

Splashscreen fixes #851

wants to merge 2 commits into from

Conversation

80avin
Copy link

@80avin 80avin commented Feb 20, 2022

Fixes

  1. prevent splashscreen from starting alwaysOnTop
  2. prevent pointer events like drag on splash screen

Note: I saw #717 but the approach is different.

@CLAassistant
Copy link

CLAassistant commented Feb 20, 2022 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@per1234 per1234 added the topic: code Related to content of the project itself label Feb 20, 2022
This was linked to issues Feb 20, 2022
Splash Screen Should not be Always on Top #324
Closed
Splash Screen Can be Clicked and Dragged as an Image #325
Closed
@AlbyIanna AlbyIanna requested review from fstasi, AlbyIanna and ubidefeo and removed request for fstasi February 24, 2022 10:55
@fstasi
Copy link
Contributor

fstasi commented Feb 24, 2022 •
edited
Loading

Hi @80avin ,

thanks for contributing, we definitely need some work on the splash screen.
I tested on mac and I see some behavior we should try to address:

  • the splash screen pops up
  • it is immediateley covered by the IDE main window (blank)

I'd get rid of the splash screen, we don't need it anymore.

@80avin let me know if this is something that can be addressed in this PR

@Logicer16
Copy link

Logicer16 commented Mar 3, 2022 •
edited
Loading

  • the splash screen pops up
  • it is immediateley covered by the IDE main window (blank)

@80avin I know setting alwaysOnTop to false would be far simpler, but this is why #717 takes a different approach.

I've also noticed that using pointer-events: none; still allows the splash screen to be highlighted if it's double clicked.

@Logicer16 Logicer16 mentioned this pull request Jul 26, 2022
Closed
@kittaakos kittaakos self-assigned this Aug 3, 2022
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
334b52b 
Added a bare minimum example.

Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
@kittaakos kittaakos mentioned this pull request Aug 4, 2022
Merged
4 tasks
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
ed97fc8 
Added a bare minimum example.

Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
0e4a6a6 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
f7a1353 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
0adde9a 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 4, 2022
Replaced the splash screen with a preload.
db9037d 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 5, 2022
Replaced the splash screen with a preload.
d956c02 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 5, 2022
Replaced the splash screen with a preload.
b967c53 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 5, 2022
Replaced the splash screen with a preload.
ecfa65f 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 5, 2022
Replaced the splash screen with a preload.
0881106 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 9, 2022
Replaced the splash screen with a preload.
bacd82f 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 9, 2022
Replaced the splash screen with a preload.
f78d85d 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 9, 2022
Replaced the splash screen with a preload.
213d764 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
kittaakos pushed a commit that referenced this pull request Aug 9, 2022
Replaced the splash screen with a preload.
18f2ff6 
Added a bare minimum example.

Closes #193
Closes #324
Closes #327
Closes #717
Closes #851

Signed-off-by: Akos Kitta <[email protected]>
@per1234 per1234 added the conclusion: duplicate Has already been submitted label Aug 9, 2022
kittaakos pushed a commit to kittaakos/arduino-ide that referenced this pull request Aug 10, 2022
@kittaakos
Replaced the splash screen with a preload.
92bc5ec 
Added a bare minimum example.

Closes arduino#193
Closes arduino#324
Closes arduino#327
Closes arduino#717
Closes arduino#851

Signed-off-by: Akos Kitta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlbyIanna AlbyIanna Awaiting requested review from AlbyIanna

@ubidefeo ubidefeo Awaiting requested review from ubidefeo

Assignees

@kittaakos kittaakos

@AlbyIanna AlbyIanna

Labels
conclusion: duplicate Has already been submitted topic: code Related to content of the project itself
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Splash Screen Can be Clicked and Dragged as an Image Splash Screen Should not be Always on Top
7 participants
@80avin @CLAassistant @fstasi @Logicer16 @kittaakos @AlbyIanna @per1234