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

Screenshot key has long "cooldown" #360

Closed
only-a-ptr opened this Issue Aug 5, 2015 · 6 comments

Comments

Projects
None yet
5 participants
@only-a-ptr
Member

only-a-ptr commented Aug 5, 2015

Context: When you take a screenshot (PrintScreen key by default), a notice box "Screenshot saved" appears, stays visible for a while and then fades out.

Problem: You cannot take another screenshot while the "screenshot saved" notice is visible. You must vait until the notice box fades out. Only then the PrintScreen key reacts again.

Test setup
RoR 0.4.5 test build 5
Windows 7
Map: auriga; Tested while driving a vehicle (Gavril MZ2R)
Parallel-split shadow maps enabled, maximum quality.

@only-a-ptr only-a-ptr added the bug label Aug 5, 2015

@only-a-ptr only-a-ptr added this to the Post-Nextstable milestone Aug 5, 2015

@Max98

This comment has been minimized.

Show comment
Hide comment
@Max98
Contributor

Max98 commented Aug 5, 2015

@only-a-ptr

This comment has been minimized.

Show comment
Hide comment
@only-a-ptr

only-a-ptr Aug 5, 2015

Member

That's only part of the problem.
The primary issue is that the notification box blocks screenshotting at all.

Member

only-a-ptr commented Aug 5, 2015

That's only part of the problem.
The primary issue is that the notification box blocks screenshotting at all.

@Max98

This comment has been minimized.

Show comment
Hide comment
@Max98

Max98 Aug 5, 2015

Contributor

Oh, so it blocks screenshoting? I thought that it just annoys the screenshot, I might take a look into this.

Contributor

Max98 commented Aug 5, 2015

Oh, so it blocks screenshoting? I thought that it just annoys the screenshot, I might take a look into this.

@DirtGamer301

This comment has been minimized.

Show comment
Hide comment
@DirtGamer301

DirtGamer301 Aug 9, 2015

I haven't tried TB5 yet as I'm still browsing through the activity in the forum and here which happaned while I was gone, but atleast in TB 4.1 I can take screenshots when the notification box appears, but it annoys :p
screenshot_4830
screenshot_4831
screenshot_4832

DirtGamer301 commented Aug 9, 2015

I haven't tried TB5 yet as I'm still browsing through the activity in the forum and here which happaned while I was gone, but atleast in TB 4.1 I can take screenshots when the notification box appears, but it annoys :p
screenshot_4830
screenshot_4831
screenshot_4832

@Hiradur Hiradur modified the milestones: Post-Nextstable, 0.4.6.0 Oct 24, 2015

@only-a-ptr only-a-ptr self-assigned this Nov 18, 2015

@ulteq

This comment has been minimized.

Show comment
Hide comment
@ulteq

ulteq Jan 19, 2016

Contributor

Two possible solutions:

  • Don't display the notification, but play a sound instead
  • Auto hide the notification right before you take another screenshot

I would first try the second one.

Contributor

ulteq commented Jan 19, 2016

Two possible solutions:

  • Don't display the notification, but play a sound instead
  • Auto hide the notification right before you take another screenshot

I would first try the second one.

@only-a-ptr

This comment has been minimized.

Show comment
Hide comment
@only-a-ptr

only-a-ptr Jan 19, 2016

Member

@ulteq Go for the second one - straightforward and clean enough. Thanks.

EDIT: Also take a look at these please (related): #473 #41

Member

only-a-ptr commented Jan 19, 2016

@ulteq Go for the second one - straightforward and clean enough. Thanks.

EDIT: Also take a look at these please (related): #473 #41

@only-a-ptr only-a-ptr assigned ulteq and unassigned only-a-ptr Jan 19, 2016

ulteq added a commit to ulteq/rigs-of-rods that referenced this issue Jan 30, 2016

[Codechange] Some screenshot taking improvements
* Reduced the screenshot key bounce time by half

* Auto hide active GUI notifications before a screenshot is taken

* Added timestamps to the screenshot filename

Fixes: RigsOfRods#360

Fixes: RigsOfRods#473
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment