gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

refactor(editor): improve naming and comments (#43662)

Browse Source 
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
Co-authored-by: Sem Bauke <46919888+Sembauke@users.noreply.github.com>
This commit is contained in:
Oliver Eyton-Williams
2021-10-04 15:48:57 +02:00
committed by GitHub
parent f5de80dc4f
commit 38d65f95b4
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
client/src/templates/Challenges/classic/editor.tsx
View File

@ -292,8 +292,8 @@ const Editor = (props: EditorProps): JSX.Element => {
editorRef.current = editor;
data.editor = editor;
if (isProject()) {
initializeProjectFeatures();
if (isProjectStep()) {
initializeProjectStepFeatures();
addContentChangeListener();
showEditableRegion(editor);
}
@ -646,7 +646,7 @@ const Editor = (props: EditorProps): JSX.Element => {
}
};
function initializeProjectFeatures() {
function initializeProjectStepFeatures() {
const editor = data.editor;
if (editor) {
initializeRegions(getEditableRegionFromRedux());
@ -654,7 +654,9 @@ const Editor = (props: EditorProps): JSX.Element => {
}
}
function isProject() {
// Currently, only practice project parts have editable region markers
// This function is used to enable multiple editor tabs, jaws, etc.
function isProjectStep() {
const editableRegionBoundaries = getEditableRegionFromRedux();
return editableRegionBoundaries.length === 2;
}
@ -957,14 +959,14 @@ const Editor = (props: EditorProps): JSX.Element => {
const { editor } = data;
const hasChangedContents = updateEditorValues();
if (hasChangedContents && isProject()) {
initializeProjectFeatures();
if (hasChangedContents && isProjectStep()) {
initializeProjectStepFeatures();
updateDescriptionZone();
updateOutputZone();
}
editor?.focus();
if (isProject() && editor) {
if (isProjectStep() && editor) {
showEditableRegion(editor);
}
// eslint-disable-next-line react-hooks/exhaustive-deps