Skip to content
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

fixed #2020 (mondo link) + added vs-code suggested extensions. #2036

Merged
merged 4 commits into from
Aug 27, 2019

Conversation

evans-g-crsj
Copy link
Contributor

What was asked is done but there are a few things I would like to to do to clean some things up.

Such as:

  • extract css in an external file

  • remove some of the {...props}

  • multiple links are not correctly aligned

  • test NCITLINK (regex could potentially be improved to avoid an extra manipulation)

…w bug => removed duplicate links in the very same column + extracted (some) css in an external file + added some props in an explict way instead of doing the evil {...props}
@evans-g-crsj evans-g-crsj marked this pull request as ready for review August 26, 2019 15:24
@evans-g-crsj
Copy link
Contributor Author

To Summarize,

  • multi mondo links can now be clicked (main bug to fix)

  • repeating links are removed (new bug found while working on the main fix)

  • multi mondo links are vertically aligned (layout fix)

  • code clean-up

Copy link
Contributor

@nyanofthemoon nyanofthemoon left a comment

Choose a reason for hiding this comment

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

A couple of questions and comments.

Copy link
Contributor

@nyanofthemoon nyanofthemoon left a comment

Choose a reason for hiding this comment

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

So cleanz! Approved.

@evans-g-crsj evans-g-crsj merged commit 77f659a into dev Aug 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants