Skip to content

Conversation

@ykhadnevich
Copy link
Collaborator

@ykhadnevich ykhadnevich commented Dec 17, 2023

Closes #60

Copy link
Contributor

@Bardin08 Bardin08 left a comment

Choose a reason for hiding this comment

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

In general LGTM 👍

Comment on lines +5 to +20
//import RichTextEditor from "../../pages/RichTextEditor";

// Example using RichTextEditor:
/*const YourPageComponent: React.FC = () => {
const handleFileUpload = (fileContent: string) => {
// Do something with the file content, e.g., save it or display it
console.log('File content:', fileContent);
};
return (
<div>
<h1>Your Page</h1>
<RichTextEditor onFileUpload={handleFileUpload} />
</div>
);
};*/
Copy link
Contributor

Choose a reason for hiding this comment

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

no need to keep it here. We use VCS to avoid keeping unused code at the codebase

Comment on lines +20 to +23
return () => {
quill.off('text-change', () => {});
quill.off('selection-change', () => {});
};
Copy link
Contributor

Choose a reason for hiding this comment

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

Why u decided to add dummy subs?

const fileContent = e.target?.result as string;
onFileUpload(fileContent);
};
reader.readAsText(file);
Copy link
Contributor

Choose a reason for hiding this comment

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

Which markdown langs are supported?

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.

Add possiblity to provide Rich Formatted Text File

4 participants