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

Rich text viewer | Implementation of markdown parser #1335

Merged
Merged
Show file tree
Hide file tree
Changes from 38 commits
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
5a348d0
Implemented markdown parser and added markdown value API for viewer c…
vivinkrishna-ni Jun 30, 2023
53bc3ce
Change files
vivinkrishna-ni Jun 30, 2023
590a2c8
Removed tiptap/pm and added only pm-markdown and pm-model as some pac…
vivinkrishna-ni Jun 30, 2023
68b157c
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jun 30, 2023
698f22b
Resolved some of the PR comments
vivinkrishna-ni Jul 4, 2023
8b955cb
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 4, 2023
f625d25
Resolve linting errors
vivinkrishna-ni Jul 4, 2023
5db53e9
Added node polyfills to support punycode used in the markdown-it library
vivinkrishna-ni Jul 4, 2023
c7d4d23
Added append document fragment when setting the markdown value
vivinkrishna-ni Jul 5, 2023
b503590
Cleared the inner HTML and appended the rich text content to viewer c…
vivinkrishna-ni Jul 5, 2023
7d1b9ee
Run npm install in the root directory
vivinkrishna-ni Jul 7, 2023
a713cfd
Updated UI and added storybook matrix tests
vivinkrishna-ni Jul 10, 2023
bb0d76a
Resolved some of the PR comments
vivinkrishna-ni Jul 10, 2023
41531b7
Updated the styles for nodes in viewer component
vivinkrishna-ni Jul 10, 2023
09cc91e
Renamed 'markdown' and removed `fit-to-content` attribute.
vivinkrishna-ni Jul 12, 2023
4a421cf
Resolve linting errors
vivinkrishna-ni Jul 12, 2023
4ee7a61
Changes for resolving some of the comments
vivinkrishna-ni Jul 12, 2023
0704f4c
Resolving lint error
vivinkrishna-ni Jul 12, 2023
18061a9
Moved the type/markdown-it to dependencies from dev
vivinkrishna-ni Jul 12, 2023
68298ce
Fixed issue in setting the margin style for list paragraphs
vivinkrishna-ni Jul 12, 2023
035e583
Resolve linting error
vivinkrishna-ni Jul 12, 2023
e5cff56
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 13, 2023
b579ce0
Updated the name of the viewer function and removed id
vivinkrishna-ni Jul 14, 2023
d4d9fe1
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 14, 2023
a549149
Test commit to check whether methods work on Safari browsers
vivinkrishna-ni Jul 14, 2023
8533518
Revoked the previous test commit
vivinkrishna-ni Jul 14, 2023
1246017
Resolved some of the PR comments
vivinkrishna-ni Jul 17, 2023
9a0b11e
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 17, 2023
5816a66
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 18, 2023
edbec0a
Added aftereach in the unit test cases
vivinkrishna-ni Jul 18, 2023
ff08921
Added the null assertion for markdown observable
vivinkrishna-ni Jul 18, 2023
770a349
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 19, 2023
c520616
Updated and added few test cases for markdown strings. Updated the sp…
vivinkrishna-ni Jul 19, 2023
5757830
Updated few minor changes.
vivinkrishna-ni Jul 19, 2023
351cd1e
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 20, 2023
8eff4f6
Updated package-lock.json
vivinkrishna-ni Jul 20, 2023
a29a7f9
Changes for resolving some comments
vivinkrishna-ni Jul 20, 2023
14ab812
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 21, 2023
426ee92
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 21, 2023
0b22333
Enabled normalize setting back and updated test cases accordingly.
vivinkrishna-ni Jul 21, 2023
9927fc8
Merge branch 'users/vivin/viewer-markdown-parser-implementation' of h…
vivinkrishna-ni Jul 21, 2023
71f520f
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 21, 2023
247c156
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 22, 2023
e3d9d40
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 24, 2023
bc69fb9
Merge branch 'main' into users/vivin/viewer-markdown-parser-implement…
vivinkrishna-ni Jul 25, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
vivinkrishna-ni marked this conversation as resolved.
Show resolved Hide resolved
vivinkrishna-ni marked this conversation as resolved.
Show resolved Hide resolved
rajsite marked this conversation as resolved.
Show resolved Hide resolved
"type": "patch",
vivinkrishna-ni marked this conversation as resolved.
Show resolved Hide resolved
"comment": "Implementation of markdown parser for converting markdown input to rich text content in the viewer",
vivinkrishna-ni marked this conversation as resolved.
Show resolved Hide resolved
"packageName": "@ni/nimble-components",
"email": "[email protected]",
"dependentChangeType": "patch"
}
jattasNI marked this conversation as resolved.
Show resolved Hide resolved
197 changes: 152 additions & 45 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 5 additions & 0 deletions packages/nimble-components/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,14 @@
"@types/d3-scale": "^4.0.2",
"@types/d3-selection": "^3.0.0",
"@types/d3-zoom": "^3.0.0",
"@types/markdown-it": "^12.2.3",
"d3-array": "^3.2.2",
"d3-random": "^3.0.1",
"d3-scale": "^4.0.2",
"d3-selection": "^3.0.0",
"d3-zoom": "^3.0.0",
vivinkrishna-ni marked this conversation as resolved.
Show resolved Hide resolved
"prosemirror-markdown": "^1.11.0",
"prosemirror-model": "^1.19.2",
"tslib": "^2.2.0"
},
"devDependencies": {
Expand All @@ -82,6 +85,7 @@
"@ni/eslint-config-javascript": "^4.2.0",
"@ni/eslint-config-typescript": "^4.2.0",
"@rollup/plugin-commonjs": "^24.0.1",
"@rollup/plugin-json": "^6.0.0",
"@rollup/plugin-node-resolve": "^15.0.1",
"@rollup/plugin-replace": "^5.0.1",
"@rollup/plugin-terser": "^0.4.0",
Expand Down Expand Up @@ -125,6 +129,7 @@
"puppeteer": "^10.1.0",
"remark-gfm": "^3.0.1",
"rollup": "^3.10.1",
"rollup-plugin-polyfill-node": "^0.12.0",
"rollup-plugin-sourcemaps": "^0.6.3",
"storybook": "^7.0.4",
"ts-loader": "^9.2.5",
Expand Down
20 changes: 18 additions & 2 deletions packages/nimble-components/rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ import resolve from '@rollup/plugin-node-resolve';
import sourcemaps from 'rollup-plugin-sourcemaps';
import terser from '@rollup/plugin-terser';
import replace from '@rollup/plugin-replace';
import json from '@rollup/plugin-json';
import nodePolyfills from 'rollup-plugin-polyfill-node';

const umdDevelopmentPlugin = () => replace({
'process.env.NODE_ENV': JSON.stringify('development')
Expand All @@ -21,7 +23,14 @@ export default [
format: 'iife',
sourcemap: true
},
plugins: [umdDevelopmentPlugin(), sourcemaps(), resolve(), commonJS()]
plugins: [
umdDevelopmentPlugin(),
nodePolyfills(),
sourcemaps(),
resolve(),
commonJS(),
json()
]
},
{
input: 'dist/esm/all-components.js',
Expand All @@ -37,6 +46,13 @@ export default [
})
]
},
plugins: [umdProductionPlugin(), sourcemaps(), resolve(), commonJS()]
plugins: [
umdProductionPlugin(),
nodePolyfills(),
sourcemaps(),
resolve(),
commonJS(),
json()
]
}
];
Loading