Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 136ec47

Browse files
committed
Code review: tweak wording of comment
1 parent 6beff76 commit 136ec47

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/editor/CSSInlineEditor.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -160,9 +160,9 @@ define(function (require, exports, module) {
160160
*
161161
* @param {!Editor} editor
162162
* @param {!{line:Number, ch:Number}} pos
163-
* @return {?$.Promise} synchronously resolved with an InlineWidget, or
164-
* {string} if pos is in tag but not in tag name, class attr, or id attr, or
165-
* null if we're not even close to a context where we could provide anything.
163+
* @return {?$.Promise} synchronously resolved with an InlineWidget; or error
164+
* {string} if pos is in tag but not in tag name, class attr, or id attr; or null if the
165+
* selection isn't even close to a context where we could provide anything.
166166
*/
167167
function htmlToCSSProvider(hostEditor, pos) {
168168

src/extensions/default/JavaScriptQuickEdit/main.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,7 @@ define(function (require, exports, module) {
188188
* @param {!Editor} editor
189189
* @param {!{line:number, ch:number}} pos
190190
* @return {$.Promise} a promise that will be resolved with an InlineWidget
191-
* or null if we're not going to provide anything.
191+
* or null if we're not ready to provide anything.
192192
*/
193193
function javaScriptFunctionProvider(hostEditor, pos) {
194194
// Only provide a JavaScript editor when cursor is in JavaScript content

0 commit comments

Comments
 (0)