Drutol / MALClient

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

Stuff I noticed (There's a lot) 🦊 #121

Closed FoxInFlame closed 7 years ago

FoxInFlame commented 7 years ago

New report will be added as a reply to this issue. Please do not close this issue. A bug report might contain replies to previous reports. I'm not really in the place to say all this (caus all I've done is like scrape small pages and create a tiny Chrome Extension) but I like reporting bugs. :)

Icons:

Drutol, you are free to edit to reply.

2017-04-09 ~09:00 GMT [Android]

Report No. 1 http://imgur.com/a/m3DYZ

Putting list here so you can edit&reply if you want :)

Drutol commented 7 years ago

Added list with replies.

FoxInFlame commented 7 years ago

Added more below. I'm sorry for the lack of screenshots, it just takes ages to send it on Google Drive to sync and upload to Imgur.

FoxInFlame commented 7 years ago

Great, I got flagged for posting a long feedback report. :)

FoxInFlame commented 7 years ago

2017-04-09 19:45 GMT [Android]

Report No. 2 I know this is a lot. Bear with me, one by one. :)

That's probably it for now. I need sleep.

FoxInFlame commented 7 years ago

2017-04-11 08:58 GMT [Android]

Report No. 3

FoxInFlame commented 7 years ago

2017-04-14 15:47 GMT [Android]

Report No. 4

Drutol commented 7 years ago

I'm slowly finishing #64 once I'm done I'm coming over here!

FoxInFlame commented 7 years ago

2017-04-15 02:52 GMT [Android 0.0.6.0]

Report No. 5

FoxInFlame commented 7 years ago

2017-04-19 05:45 GMT [Android 0.0.7.0]

Report No. 6

FoxInFlame commented 7 years ago

2017-04-20 05:38 GMT [Android 0.0.7.0]

Report No. 7

Drutol commented 7 years ago

Are you sure about recommendations pictures using old loading mechanism?

FoxInFlame commented 7 years ago

Um, does that image in Screenshot 2 look like a cute moe girl with purple hair?

Drutol commented 7 years ago

Well, that's strange... maybe one of these rare bugs ;d

FoxInFlame commented 7 years ago

2017-04-20 07:41 GMT [Android 0.0.7.0]

Report No. 8

Drutol commented 7 years ago

Well, it works now but it'll split itself on next run probably. Should I split input with commas to different multiple tags?

FoxInFlame commented 7 years ago

Yeah, it split on the next run. Splitting in input would be wise.

2017-04-20 13:16 GMT [Android 0.0.7.0]

Report No. 9 Forums :)

Drutol commented 7 years ago

The thing with forums is that posts' content is rendered in web view and web view content height has to be extracted using my sick javascript skills and finally applied to web view in native layout. This is rough on android, it's a little less rough on UWP version but there's still chance of this happening. This is pure voodoo.

Posts on the index page are not loading because there was change in html yesterday.

I could add an option to render it as a text without quotes and all this stuff.

As far as screen capturing goes... I don't use android, I've no clue. I'd probably share my screen to computer and than run gifcam.

Drutol commented 7 years ago

@FoxInFlame did you get any pop-ups asking whether app can use internal storage? Is data being cached properly (one loaded details load instantly)? I got some news that android 7.0 has some wicked policy going on.

FoxInFlame commented 7 years ago

It doesn't look like I did. (I know what caching is, I'm a fellow developer :])

Maybe that explains the reason my list doesn't load offline, or why images keep having flashes.

But wait, my anime list doesn't seem to have image problems, so maybe caching is working...

Drutol commented 7 years ago

By

once loaded details load instantly

I meant that's the easiest way to check whether it works.

Do you get loaders on once visited anime details page?

FoxInFlame commented 7 years ago

No. So, maybe, caching is working. That means I pressed Ok on a popup that I don't remember...

Drutol commented 7 years ago

Could you try reinstall? I'm curious about that ^^

FoxInFlame commented 7 years ago

I'll do the reinstall when I get back home. Will edit later.

Edit: The first dialog upon installation only asked for permission for Photos/Media/Files and Wi-Fi connection information. If P/M/F counts as Cache, then that's great.

FoxInFlame commented 7 years ago

2017-04-21 08:15 GMT [Android 0.0.7.0]

Report No. 10

Drutol commented 7 years ago

@FoxInFlame how are notifications looking in v0.0.8.0?

FoxInFlame commented 7 years ago

2017-04-25 21:22 UTC [Android 0.0.8.0]

Report No. 11

Drutol commented 7 years ago

Notifications shouldn't duplicate, that's what I've been trying to address.

FoxInFlame commented 7 years ago

Ah. That should be fine. Fixed. :)

FoxInFlame commented 7 years ago

@Drutol By the way, did you fix this? There's a text that says 'edit', probably an scrapping error.

Drutol commented 7 years ago

Wow, how did I missed it? By the way I've fixed auto sorting order chooser because it wasn't working on android too ^^

FoxInFlame commented 7 years ago

2017-04-29 16:50 UTC [Android 0.0.8.0]

Report No. 12

FoxInFlame commented 7 years ago

2017-04-30 14:14 UTC [Android 0.0.9.0]

Report No. 13

Drutol commented 7 years ago

I'm unable to reproduce this issue with adding anime to list. The only possible scenario is that this movie was already added to your list and local one got desynchronized somehow.

As for reverting to default sorting, this is intended behaviour navigating back keeps order as it was but every navigation to "fresh" list page results in it setting sorting as per settings.

FoxInFlame commented 7 years ago

God, I just viewed my profile About Me on the UWP version, and it looked horrible ;) All the whitespace, black text on gray background, block images instead of inline, ugh.

2017-05-01 23:19 UTC [UWP 5.3.14.0]

Report No. 14

Drutol commented 7 years ago

Reviews are fixed, since android 0.0.9.0, it's in shared logic. I just need to release update somewhen.

As for "about me" nothing I can do about this, there's tweaked stylesheet prepared but it's overridden directly from this html........................ ...... ...................

FoxInFlame commented 7 years ago

Mmhmkay.

2017-05-06 [Android 0.0.11.0]

Report No. 15

FoxInFlame commented 7 years ago

Just read this, and seemed interesting. If this gets implemented, wouldn't developing for Android from UWP be easier?

https://www.theverge.com/platform/amp/2017/5/11/15621428/microsoft-cross-device-ios-android-apps

Drutol commented 7 years ago

Nope, these are just new features that glue cross platform apps. There's a framework that allows you to develop UI on all 3 platforms simultaneously (Xamarin Forms) but it has long way to go.

Build17 stuff is exciting nevertheless. Especially this fluent design looks nifty.

FoxInFlame commented 7 years ago

2017-05-21 08:20 UTC [Android 0.0.15.0]

Report No. 16

FoxInFlame commented 7 years ago

2017-06-19 21:54 UTC [Android 1.0.1.1]

Report No. 17

Drutol commented 7 years ago

My versioning goes like this:

(hey, I've rewritten majority of an app).(oh, new big feature).(random minor things).(I've messed up and this is a crashfix)

And 1.0.0.0 just means release.

FoxInFlame commented 7 years ago

2017-06-27 14:53 UTC [Android 1.0.3.1]

Report No. 18