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.

Sign up for GitHub

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

Jump to bottom

Core Enhancement 2 -Incorporation of Modules in the Articles-Edit-View #8

Open
wants to merge 36 commits into
base: dev
Choose a base branch
from
Open

Core Enhancement 2 -Incorporation of Modules in the Articles-Edit-View #8

wants to merge 36 commits into from
+252 −0

Conversation

YatharthVyas
Copy link
Collaborator

@YatharthVyas YatharthVyas commented Jun 29, 2021 •
edited
Loading

The purpose of this PR is to track all the file changes and commits made. Mentors can review the changes and provide feedback.
I will update the description as and when I push more commits.

Plan Document: https://docs.google.com/document/d/1Pl8JGa2hkYkmJzQOn9_mS8a4imDmqc2a/edit#heading=h.xzcjfqetxfhr

  • Switcher in Article's Config Form to toggle the Imported Modules Tab
  • New Tab in Article-Edit View for Imported Modules
  • New Button to Create New Module directly from Article Edit View using an editor-xtd Plugin (IN PROGRESS)

Summary of Changes

Option to Enable the Tab via Article Edit View's Configure Edit Screen Tab

image

Option to Enable the Tab via Article's Global Config From (Under Editing Layout Tab)

image

The Imported Modules Tab

image

New Module Editor-xtd Button

  1. For TinyMCE

image

  1. For CodeMirror

image

@YatharthVyas
Copy link
Collaborator Author

modal

@YatharthVyas
Copy link
Collaborator Author

Modules can now be removed from the Imported Modules Tab

remove

@YatharthVyas
Copy link
Collaborator Author

YatharthVyas commented Jul 8, 2021 •
edited
Loading

new

To Do:

  • Remove the plugin files for the New Module Plugin: Done

@YatharthVyas YatharthVyas mentioned this pull request Jul 12, 2021
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richard67 richard67 richard67 left review comments

@chmst chmst chmst left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YatharthVyas @chmst @richard67