Bug 426685

Summary: Dual screen setup out of bounds area has content in screenshot
Product: [Applications] Spectacle Reporter: MX <max.neupert>
Component: GeneralAssignee: Boudhayan Gupta <me>
Status: RESOLVED FIXED    
Severity: normal CC: justin.zobel, kde
Priority: NOR    
Version First Reported In: 19.12.3   
Target Milestone: ---   
Platform: Kubuntu   
OS: Linux   
URL: https://twitter.com/ChairAudio/status/1306941934062260225
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Screenshot

Description MX 2020-09-18 13:17:22 UTC 
SUMMARY


STEPS TO REPRODUCE
1. have a dual screen setup with mismatching resolutions (one screen has less pixels in height in a horizontal setup)
2. do a full screenshot of the combined area (which will contain black space for the smaller screen

OBSERVED RESULT
The area which should be blank has pixel content from the buffer

EXPECTED RESULT
The mismatching area should be blank

SOFTWARE/OS VERSIONS
Operating System: Kubuntu 20.04
KDE Plasma Version: 5.18.5
KDE Frameworks Version: 5.68.0
Qt Version: 5.12.8
Kernel Version: 5.4.65-050465-lowlatency
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4702MQ CPU @ 2.20GHz
Memory: 7,7 GiB of RAM
Comment 1 Justin Zobel 2020-11-22 09:22:57 UTC 
Thanks for the report MX. 

I've just tested this with spectacle 20.08.3 and I can't replicate the issue. I'll attach a screenshot.

Can you please upload one so we can see what the issue is that you're facing?
Comment 2 Justin Zobel 2020-11-22 09:23:23 UTC 
Created attachment 133550 [details]
Screenshot
Comment 3 MX 2020-11-22 11:18:07 UTC 
Can't reproduce it with now upgraded system.
To proof I wasn't hallucinating there is an older tweet:
https://twitter.com/ChairAudio/status/1306941934062260225
Comment 4 Justin Zobel 2020-11-22 21:45:41 UTC 
(In reply to MX from comment #3)
> Can't reproduce it with now upgraded system.
> To proof I wasn't hallucinating there is an older tweet:
> https://twitter.com/ChairAudio/status/1306941934062260225

Thanks for that, no proof required though. Thanks for re-testing.