Skip to content

Commit

Permalink
Add full image quality support (#22654)
Browse files Browse the repository at this point in the history
  • Loading branch information
RobsonPontin authored Dec 9, 2022
1 parent 60bf868 commit 195f14a
Showing 1 changed file with 56 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -5,18 +5,16 @@
namespace Peek.FilePreviewer.Previewers
{
using System;
using System.Collections.Generic;
using System.Drawing.Imaging;
using System.IO;
using System.Threading;
using System.Threading.Tasks;
using CommunityToolkit.Mvvm.ComponentModel;
using Microsoft.UI.Dispatching;
using Microsoft.UI.Xaml.Media.Imaging;
using Peek.Common;
using Peek.Common.Extensions;
using Windows.Foundation;
using Windows.Graphics.Imaging;
using Windows.Storage.Streams;
using Windows.Storage;
using File = Peek.Common.Models.File;

public partial class PngPreviewer : ObservableObject, IBitmapPreviewer
Expand Down Expand Up @@ -44,10 +42,16 @@ public PngPreviewer(File file)

private DispatcherQueue Dispatcher { get; }

private Task<bool>? PreviewQualityThumbnailTask { get; set; }

private Task<bool>? FullQualityImageTask { get; set; }

private CancellationTokenSource _cancellationTokenSource = new CancellationTokenSource();

private CancellationToken CancellationToken => _cancellationTokenSource.Token;

private bool IsFullImageLoaded => FullQualityImageTask?.Status == TaskStatus.RanToCompletion;

public void Dispose()
{
_cancellationTokenSource.Dispose();
Expand All @@ -69,9 +73,10 @@ public async Task LoadPreviewAsync()
{
State = PreviewState.Loading;

var previewTask = LoadPreviewImageAsync();
PreviewQualityThumbnailTask = LoadPreviewImageAsync();
FullQualityImageTask = LoadFullImageAsync();

await Task.WhenAll(previewTask);
await Task.WhenAll(PreviewQualityThumbnailTask, FullQualityImageTask);

if (Preview == null)
{
Expand All @@ -95,23 +100,62 @@ private void OnPropertyChanged(object? sender, System.ComponentModel.PropertyCha
}
}

private Task LoadPreviewImageAsync()
private Task<bool> LoadPreviewImageAsync()
{
var thumbnailTCS = new TaskCompletionSource();
Dispatcher.TryEnqueue(async () =>
return TaskExtension.RunSafe(async () =>
{
if (CancellationToken.IsCancellationRequested)
{
_cancellationTokenSource = new CancellationTokenSource();
return;
}
Preview = await ThumbnailHelper.GetThumbnailAsync(File.Path, _png_image_size);
thumbnailTCS.SetResult();
if (!IsFullImageLoaded)
{
await Dispatcher.RunOnUiThread(async () =>
{
Preview = await ThumbnailHelper.GetThumbnailAsync(File.Path, _png_image_size);
});
}
});
}

private Task<bool> LoadFullImageAsync()
{
var thumbnailTCS = new TaskCompletionSource();
return TaskExtension.RunSafe(async () =>
{
if (CancellationToken.IsCancellationRequested)
{
_cancellationTokenSource = new CancellationTokenSource();
return;
}
return thumbnailTCS.Task;
await Dispatcher.RunOnUiThread(async () =>
{
WriteableBitmap? bitmap = null;
var sFile = await StorageFile.GetFileFromPathAsync(File.Path);
using (var randomAccessStream = await sFile.OpenStreamForReadAsync())
{
// Create an encoder with the desired format
var decoder = await BitmapDecoder.CreateAsync(
BitmapDecoder.PngDecoderId,
randomAccessStream.AsRandomAccessStream());
var softwareBitmap = await decoder.GetSoftwareBitmapAsync(
BitmapPixelFormat.Bgra8,
BitmapAlphaMode.Premultiplied);
// full quality image
bitmap = new WriteableBitmap((int)decoder.PixelWidth, (int)decoder.PixelHeight);
softwareBitmap?.CopyToBuffer(bitmap.PixelBuffer);
}
Preview = bitmap;
});
});
}
}
}

1 comment on commit 195f14a

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@check-spelling-bot Report

🔴 Please review

See the 📜action log for details.

Unrecognized words (140)
Abbrivation
apidl
ari
arw
BESTEFFORT
BHIDSF
BVal
calpwstr
CARRAY
CElems
Chromakey
cidl
crw
CVal
DANGEROUSLYCOMMITMERELYTODISKCACHE
dcr
dcs
Dct
DDEIf
Dds
DELAYCREATION
drf
Dwma
eip
Exa
exabyte
Excep
EXCEPINFO
EXTRINSICPROPERTIES
EXTRINSICPROPERTIESONLY
FASTPROPERTIESONLY
filetime
HANDLERPROPERTIESONLY
HDR
hicon
hif
HVal
IBitmap
IBlock
IColor
icolumn
IContext
IDecoder
IEncoder
IEnum
IIDI
iiq
IMetadata
IPalette
IQuery
IReader
ISource
ISurface
ithumbnail
jfi
jif
kdc
Keybd
Lcid
LOCKBYTES
LOCKTYPE
LVal
mdc
mef
Mega
mrw
neighborings
NOOPEN
nrw
ONLYIFCURRENT
ONLYONCE
OPENSLOWITEM
openspecs
OPLOCK
ori
overriden
pbgra
PBlob
pcch
pcelt
pcs
pef
PElems
Percision
petabyte
pkey
ppenum
pprop
PREFERQUERYPROPERTIES
Previer
PRGBA
PROPERTYNOTFOUND
PROPVARIANT
pscid
psfi
pstatstg
pstm
pui
pvar
raf
retunred
rfid
RGBE
rgelt
rgf
rwl
rwz
sachaple
SAFEARRAY
SCID
Scode
Shcontf
SHELLDETAILS
Shgno
Softcoded
srf
SRGB
STGC
STGTY
Stroe
Strret
tcs
terabyte
titlebar
tlbimp
toogle
UMsg
UOffset
USERDEFINED
UType
VARTYPE
VERSIONED
webbrowser
windowsapp
WMSDK
WReserved
WScan
wsp
WVk
YQuantized
Previously acknowledged words that are now absent brucelindbloom chromaticities companding DCR Eqn ffaa FILETIME HICON ITHUMBNAIL Pbgra PKEY Windowsapp :arrow_right:
To accept ✔️ these unrecognized words as correct and remove the previously acknowledged and now absent words, run the following commands

... in a clone of the git@github.com:microsoft/PowerToys.git repository
on the peek branch (ℹ️ how do I use this?):

curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/v0.0.21/apply.pl' |
perl - 'https://github.com/microsoft/PowerToys/actions/runs/3653696059/attempts/1'
Available 📚 dictionaries could cover words not in the 📘 dictionary

This includes both expected items (2140) from .github/actions/spell-check/expect.txt and unrecognized words (140)

Dictionary Entries Covers
cspell:win32/src/win32.txt 53509 133
cspell:cpp/src/cpp.txt 30216 129
cspell:python/src/python/python-lib.txt 3873 32
cspell:php/php.txt 2597 17
cspell:node/node.txt 1768 14
cspell:typescript/typescript.txt 1211 12
cspell:python/src/python/python.txt 453 11
cspell:java/java.txt 7642 11
cspell:aws/aws.txt 218 8
cspell:r/src/r.txt 808 7

Consider adding them using (in .github/workflows/spelling2.yml):

      with:
        extra_dictionaries:
          cspell:win32/src/win32.txt
          cspell:cpp/src/cpp.txt
          cspell:python/src/python/python-lib.txt
          cspell:php/php.txt
          cspell:node/node.txt
          cspell:typescript/typescript.txt
          cspell:python/src/python/python.txt
          cspell:java/java.txt
          cspell:aws/aws.txt
          cspell:r/src/r.txt

To stop checking additional dictionaries, add:

      with:
        check_extra_dictionaries: ''
Errors (1)

See the 📜action log for details.

❌ Errors Count
❌ forbidden-pattern 1

See ❌ Event descriptions for more information.

If the flagged items are 🤯 false positives

If items relate to a ...

  • binary file (or some other file you wouldn't want to check at all).

    Please add a file path to the excludes.txt file matching the containing file.

    File paths are Perl 5 Regular Expressions - you can test yours before committing to verify it will match your files.

    ^ refers to the file's path from the root of the repository, so ^README\.md$ would exclude README.md (on whichever branch you're using).

  • well-formed pattern.

    If you can write a pattern that would match it,
    try adding it to the patterns.txt file.

    Patterns are Perl 5 Regular Expressions - you can test yours before committing to verify it will match your lines.

    Note that patterns can't match multiline strings.

Please sign in to comment.