Drutol / MALClient

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

UI doesn't switch to Mobile view when resized low enough #12

Closed AzureKitsune closed 8 years ago

AzureKitsune commented 8 years ago

The UI should switch to the mobile view when you resize the window low enough. Otherwise, the anime info pane overlaps the anime list. mal

Drutol commented 8 years ago

This works as intended right now. Mobile and desktop views are so much different that it would take much more time to merge than it's worth imo. I may remove this minimum width value for items grid.

AzureKitsune commented 8 years ago

Do you accept pull requests? Shouldn't be too hard with the RelativePanel.

Drutol commented 8 years ago

If it's going to work better , sure thing :)

AzureKitsune commented 8 years ago

Heh, you weren't kidding about the UIs being extremely different. I am gonna have to recreate the UI. 😭

Drutol commented 8 years ago

Yeah, it's because of this splitter thingy. ^^

AzureKitsune commented 8 years ago

This issue still bothers me a bit. I feel like you're doubling work for yourself by maintaining two branches. I will still submit a PR in the future to help out a bit.

Drutol commented 8 years ago

Yeah, it's a bit troublesome but combining this into one view would require tons of work too. I'm planning to put this into one branch with some shared resources but I can't get myself to do that :)

AzureKitsune commented 8 years ago

I'll throw up a PR now just to get the ball rolling. It'll be a while before I can get things going.