Skip to content

Commit b11565b

Browse files
megothTallTed
authored andcommitted
Apply suggestions from code review
Thank you for all your suggestions @TallTed ^_^ Co-Authored-By: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
1 parent 0dfaa0c commit b11565b

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/utils.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ function syncTableToArray (table, things, createNewRow) {
173173
* @param {function({NamedNode})} createNewRow(thing) returns a rendering of a new thing
174174
*
175175
* Ensures order matches exacly. We will re-rder existing elements if necessary
176-
* Can be used in fact for any element type - does not have to be a table and tr.
176+
* Can be used for any element type; does not have to be a table and tr.
177177
* Any RDF node value can only appear ONCE in the array
178178
*/
179179
function syncTableToArrayReOrdered (table, things, createNewRow) {
@@ -197,10 +197,10 @@ function syncTableToArrayReOrdered (table, things, createNewRow) {
197197
const existingRow = elementMap[thing.toNT()]
198198
if (existingRow) {
199199
table.removeChild(existingRow)
200-
table.insertBefore(existingRow, row) // Insert existing ro in place of this one
200+
table.insertBefore(existingRow, row) // Insert existing row in place of this one
201201
} else {
202202
const newRow = createNewRow(thing)
203-
row.before(newRow) // Insert existing ro in place of this one
203+
row.before(newRow) // Insert existing row in place of this one
204204
newRow.subject = thing
205205
}
206206
}

src/widgets/buttons.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -723,15 +723,15 @@ buttons.allClassURIs = function () {
723723
return set
724724
}
725725

726-
/** Figuring which propertites we know about
726+
/** Figuring which properties we know about
727727
*
728-
* When the user is inputs an RDF property, like foir a form field
729-
* or when specifying the relationshiip bteween two arbitrary things,
728+
* When the user inputs an RDF property, like for a form field
729+
* or when specifying the relationship between two arbitrary things,
730730
* then er can prompt them with properties the session knows about
731731
*
732732
* TODO: Look again by catching this somewhere. (On the kb?)
733733
* TODO: move to diff module? Not really a button.
734-
* @param {Store} kb The quadstore to be searchhed.
734+
* @param {Store} kb The quadstore to be searched.
735735
*/
736736

737737
buttons.propertyTriage = function (kb) {

0 commit comments

Comments
 (0)