-
Notifications
You must be signed in to change notification settings - Fork 202
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Overhaul] Implementation of a more consistent & modern UI #899
Conversation
…s cluttered and not a monolithic file
… back wasn't shown
…ay of showing/hiding without the need of rerendering everything
…l changes to the DecksListView
& Fixed spelling
…on into small-ui-overhaul
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for the contribution! Looks like there are some conflicts since some of the changes are reverting things that have already gone in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
Please resolve the merge conflicts (caused by merging #902)
I think this PR also fixes the bug in issue #768, as I've changed the mobile view such that this shouldn't be a problem anymore, but I don't have a IOS device, so I can't test it. |
Hello, can you please see this ticket : #1101 |
Sry I think I cant help you here, but I haven't changed anything in regards with how the markdown is rendered. I just transformed the containers around the markdown rendered content. Thus I couldn't have changed anything how an Image is rendered. |
Description
This PR is an overhaul of the main ui and it implements the issue
#872
. More detail on what the PR implements and more documentation in general can be found there.Changes