Drutol / MALClient

Not so small client app for Myanimelist.net - Windows 10 UWP & Android
GNU General Public License v3.0
352 stars 32 forks source link

Some stuff in the UWP version #122

Open Guerra24 opened 7 years ago

Guerra24 commented 7 years ago

New report will be added as a reply to this issue. Please do not close this issue. UWP edition of #121.

Icons:

Feel free to edit.


Drutol commented 6 years ago

Are you planning to blurify things?

Guerra24 commented 6 years ago

Not really, do some minor maintenance. Fix a small typo that I can't unsee and check some stuff that doesn't work right.

Guerra24 commented 6 years ago

More problems... I'm unable to launch it from VS both in Debug and Release mode, it gets stuck on splash forever, no error, nothing.

image

EDIT: Temporary workaround by deploying the app so I can test it, it will be slow and painful but is the only way ATM.

Drutol commented 6 years ago

Hmm what version of Microsoft.NETCore.UniversalWindowsPlatform nuget are you using?

Guerra24 commented 6 years ago

image 😢 😐

Guerra24 commented 6 years ago

What do you think?

image There is reveal in a lot of new places (I mean, A LOT). img image Switched to Acrylic blur for the header, increased image blur with a slightly darker color. image

So....

image image

This change requires the UWP Shared code to use FCU SDK, I tried to use conditional Xaml but it requires CU SDK so...

Extra

Where do you set the title bar color? (This is for the caption buttons background color). Calendar is broke (this is since the latest store update). Comp. view header text is also broke (this too).

EDIT: Guerra24/MALClient@2a33841874851e6b6b3a77552854521898826e29

Drutol commented 6 years ago

Wow, great job! Sorry that you had to go though this code ^^

I feel like the upper bar should be darker because now it's just "melting" with everything else. https://github.com/Drutol/MALClient/blob/master/MALClient.Dekstop/App.xaml.cs#L190

I'll try to find some time today to fix airing stuff.

Drutol commented 6 years ago

Feel free to submit pull request :)

Guerra24 commented 6 years ago

The entire app header (Title bar included)?

Okay πŸ˜ƒ

Drutol commented 6 years ago

Probably? Depends on how it looks ^^

Guerra24 commented 6 years ago

This is how it looks.

image

EDIT: Hamburger menu with separators of the same color and the bottom grid transparent. img

Drutol commented 6 years ago

Uppershadow looks nicer in my opinion.

Nice accent: image

Guerra24 commented 6 years ago

πŸ˜‚ I would prefer black but it can't be set so πŸ€·β€β™‚οΈ

I've tried to increase the Shared project min sdk to FCU but it complains about it no begin compatible with 10586, it needs that because I've used acrylic on the list elements, it works but I don't know if the store would accept that.

Drutol commented 6 years ago

I remember having some issues with that too. In the end I didn't increase it ^^ What matters most for the store is main project's configuration so it shouldn't complain.

Guerra24 commented 6 years ago

~Did you change something in the BGTasks? Now these projects refuse to compile~

😐 Cloned again the project and it worked.... wtf with vs...

Drutol commented 6 years ago

image

Guerra24 commented 6 years ago

Is there a way to style scrollbars? I've found a way to fix the ones that are behind acrylic by adding margin to top and bottom of the scrollbar.

Drutol commented 6 years ago

If I were you I wouldn't touch them. It's just pain. At least they were 1,5y ago.

Guerra24 commented 6 years ago

It was NOT really hard, just added this: img Here And... img

EDIT: Moved some elements to the titlebar.

img

EDIT2: Moved search to titlebar.

img

I will create a PR once I get all the header stuff done.

Guerra24 commented 6 years ago

Finally completed header integration into title bar. It frees up some space in the list and creates a 3-column-view on the entire window.

img

EDIT: I completely forgot about the Load all details pop-up, it works but the grid gets placed at the bottom and doesn't update.

img

Guerra24 commented 6 years ago

TL;DR Reworked again the UI.

img img img

Guerra24 commented 6 years ago

Hey! I'm back after a month (or two) of inactivity. My PC's PSU broke and took a while to get a new one, I'm going to continue working on this on my free time but meanwhile I got a small maintenance PR.

Drutol commented 6 years ago

Hello! Thanks!

Guerra24 commented 6 years ago

Got mobile fully working again, the PR is ready and both versions are OK to release (if nothing broke (I hope no)).

EDIT: HOLD ON, connected animations might have broke when I was fixing Mobile compilation, once I wake up I will fix it.

Drutol commented 6 years ago

Hehe, hmm... I haven't update mobile in ages... might be good idea ^^

Guerra24 commented 6 years ago

It seems to have been fixed on its own... Feel free to merge.

Guerra24 commented 6 years ago

I'm gonna change the target SDK to April Update (17134), are you OK with that? Also, why does Win10 Desktop notifications only show There are new notifications message instead of Win10 Mobile where it actually tells you what it is?

Drutol commented 6 years ago

No clue, I didn't notice There are new notifications yet. (I'm on April's update)

Go ahead with new SDK, I'm fighting with MSFT about certification right now...

Guerra24 commented 6 years ago

~(Some) Promotional Videos broke, android and uwp affected.~ Uh, it fixed on it's own πŸ€”πŸ€”πŸ€”

Btw, because I'm gonna continue working here and to make this more easy for both, would you give me write access to the repo? So I don't have to make PRs for single commits or similar stuff.

Drutol commented 6 years ago

Well, I don't mind. The master will be kept protected though :)

Guerra24 commented 6 years ago

Did some labels renames, examples: Android -> Platform: Android, Info -> Type: Info. And new lablels: Platform: UWP, Status: Cannot Reproduce, Type: Bug, Type: Question, etc.

Also write an issue template, what do you think?