Skip to content

Spell & Grammar Check #2281

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

Closed
devvaannsh wants to merge 4 commits into from
Closed

Spell & Grammar Check #2281

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
65ce6ab
feat: wire spell check marker api to code inspector
devvaannsh May 30, 2025
0ebf0e8
refactor: update type marker priorities
devvaannsh May 30, 2025
d58e21d
feat: working demo prototype of spell grammar check
devvaannsh May 30, 2025
f6f894f
chore: auto update API docs
devvaannsh May 30, 2025
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
9 changes: 8 additions & 1 deletion src/language/CodeInspection.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,9 @@ define(function (require, exports, module) {
/** Maintainability issue, probable error / bad smell, etc. */
WARNING: "warning",
/** Inspector unable to continue, code too complex for static analysis, etc. Not counted in err/warn tally. */
META: "meta"
META: "meta",
/** For spelling or grammatical error */
SPELL: "spell"
};

function _getIconClassForType(type, isFixable) {
Expand All @@ -98,6 +100,9 @@ define(function (require, exports, module) {
case Type.META: return isFixable ?
"line-icon-problem_type_info fa-solid fa-wrench":
"line-icon-problem_type_info fa-solid fa-info-circle";
case Type.SPELL: return isFixable ?
"line-icon-problem_type_spell fa-solid fa-wrench":
"line-icon-problem_type_spell fa-solid fa-book";
default: return isFixable ?
"line-icon-problem_type_info fa-solid fa-wrench":
"line-icon-problem_type_info fa-solid fa-info-circle";
Expand Down Expand Up @@ -430,6 +435,7 @@ define(function (require, exports, module) {
case Type.ERROR: return Editor.getMarkOptionUnderlineError();
case Type.WARNING: return Editor.getMarkOptionUnderlineWarn();
case Type.META: return Editor.getMarkOptionUnderlineInfo();
case Type.SPELL: return Editor.getMarkOptionUnderlineSpellcheck();
}
}

Expand All @@ -438,6 +444,7 @@ define(function (require, exports, module) {
case Type.ERROR: return 3;
case Type.WARNING: return 2;
case Type.META: return 1;
case Type.SPELL: return 1;
Comment thread
devvaannsh marked this conversation as resolved.
Outdated
}
}

Expand Down
Loading