Skip to content

Conversation

@breda
Copy link

@breda breda commented Jun 12, 2015

At line 97, this :
and it's not marked as read-only (writable:false)
should actually be this :
and it's not marked as read-only (writable:true)

At line 97, this :
**and it's not marked as read-only (`writable:false`)**
should actually be this : 
**and it's not marked as read-only (`writable:true`)**
@getify
Copy link
Owner

getify commented Jun 12, 2015

@breda
Copy link
Author

breda commented Jun 12, 2015

Ah! Sorry about that @getify

But, as you stated, that the (writable:false) is only associated with the word "read-only",
then I suggest adding something to clarify that, leaving it like this will probably confuse people, and/or lead to more pull-requests like this one.

@getify
Copy link
Owner

getify commented Jun 12, 2015

I am aware of the running confusion. Paradoxically, the more this issue comes up, and I have to rethink through my reasoning, the more I get convinced the original wording was correct and I'm less understanding of the other interpretation. What, grammatically, leads people to assume the parenthetical applies to the whole phrase and not just the term it immediately proceeds?

That having been said, it's already definitely something that will be addressed in the second edition. I'll use your bug specifically for tracking that. :)

@guser9822
Copy link

@getify Probably people get confused by this allegedly typo because the first sentence is in bold, differently from the second sentence. But, honestly, reading the second sentence after the first, I get confused too, and I convice myself that it's a typo

@getify getify force-pushed the master branch 6 times, most recently from 5cb284a to 8af48e2 Compare August 28, 2019 12:37
@getify getify closed this Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants