Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Patterns / Modal Dialog documentation page has errors

Open Beginner friendly
#676 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Quiet
Tech stack
javascript
Domain
documentation

Research direction

Open the Patterns / Modal Dialog documentation page and inspect the shown

hyperscript, starting with the reported tokenizer error in https://hyperscript.org/js/_hyperscript.js at line 13, column 0. Correct the example so the unterminated-string error is gone, then reload the page and confirm the console error no longer appears.

Written by the indexing model from the issue text.

Description

The modal dialog pattern page has an error

hyperscript errors were found on the following element:
<dialog _="
    on input from &lt;input,textarea/&gt; in me set :dirty to true
&lt;p&gt;on closeRequested
if :dirty
if confirm('Discard unsaved changes?')
set :dirty to false then close me
end
else
close me
end&lt;/p&gt;
&lt;p&gt;on click[target is me] send closeRequested to me
on cancel halt the event then send closeRequested to me
&quot;&gt;
&lt;div class=" modal-header"="" open="">
<h3>Edit Contact</h3>
<button _="on click send closeRequested to closest &lt;dialog/&gt;" data-hyperscript-powered="true">×</button>
</dialog> 
"

"
"Unterminated string at [Line: 13, Column: 0]"
"#consumeString@https://hyperscript.org/js/_hyperscript.js:551:24
#tokenize@https://hyperscript.org/js/_hyperscript.js:602:48
#initElement@https://hyperscript.org/js/_hyperscript.js:2799:46
@https://hyperscript.org/js/_hyperscript.js:2888:28
forEach@https://hyperscript.org/js/_hyperscript.js:2558:15
processNode@https://hyperscript.org/js/_hyperscript.js:2887:21
@https://hyperscript.org/js/_hyperscript.js:10392:29"
Image
Dominant language
JavaScript
Stars
3.8k
Forks
171
PR merge metrics
No merged PRs in 30d

Getting set up

We have not checked this project's setup files yet. Start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from bigskysoftware/_hyperscript

All issues in bigskysoftware/_hyperscript

Similar issues

More JavaScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.