Next - #160
Merged
Merged
Next#160
Conversation
- configuring: 13 API values moved from bold/italic to backticks, six
"by default" openers, three "there is a possibility to" constructions,
eight passive clauses and three "via" replaced
- customization: fixed the "calender" typo, `<style>` in backticks,
list items capitalized, number agreement in "a new CSS class(es)"
- datepicker: fixed number agreement, three missing articles and two
wrong prepositions ("attach into", "on click in"); class names in
backticks where they refer to code identifiers
- headings left untouched: features.md links to their anchors
reviewed with the article-grammar skill; each page re-run after editing,
which caught four defects introduced by the edits themselves
- fixed a missing infinitive: "Learn how initialize Calendar" - fixed prepositions and articles: "a timepicker into Calendar", "use DHTMLX Optimus framework", "creating DHTMLX-based app" - "perform the code after the component's rendering" -> "run code after the component renders" - removed four section intros that only restated their own heading - replaced five "X of Y" constructions with noun phrases and dropped the padded "necessary" in two table cells - re-run after editing caught two more: "for + gerund" in two cells and an inconsistent article in "call the Calendar events"
- section headings switched to the imperative form - API method names wrapped in code formatting instead of bold - passive constructions rewritten in active voice
- passive constructions and gerund chains rewritten in active voice - file names and the container id wrapped in code formatting - "X of Y" phrases replaced with noun phrases or active clauses - missing article added to the "Select an initial date" heading, its in-page anchor and the link to it updated together - duplicate intro sentence dropped from "Initialize Calendar"
- missing article added in "create a calendar" - mode list made parallel: "or as a timepicker" -> "or timepicker" - marketing sentence about a "great date picker solution" dropped - duplicate link to online samples removed from the intro, the one under Related resources kept - comma added after the introductory phrase in the Suite download item - "add Calendar on the page" -> "add Calendar to a page"
- typo fixed: "your need to" -> "you need to" - list intro made a complete sentence, both items capitalized - dhx.i18n.setLocale() moved from bold to code formatting - "days of a week" -> "days of the week" - dhtmlxCalendar renamed to Calendar to match the rest of the page - filler "just" and the non-idiomatic "a ready locale" dropped
- API names, types and string values moved from bold/italic to code formatting (13 occurrences) - mode values in the getCurrentMode() list aligned with the quoted form used in the showDate() list - "allows selecting" -> "allows you to select" in all three modes - commas added after leading dependent clauses - missing articles restored: "the selected date", "the date format" - "via", "and/or" and "a certain mode" replaced with direct wording - passive and future rewritten in active present: attaching to an input, linking two calendars, the setValue() return value - "There is a possibility to get" and "Please note that" dropped Section headings left as they are: renaming them would change the anchors, and 7 links across features.md, customization.md and the whatsnew.md release-notes archive point at them.
- three method names moved from bold to backticks, module name too - section headings changed from gerunds to verb phrases: "Attaching event listeners" -> "Attach event listeners" (no anchors link here, verified across the docs tree) - removed passive voice and future tense from both prose sentences - "the names of events" -> "event names", "the list of events" -> "event list" - re-run after editing flagged the three identical "Use the ... method" openers the first pass had introduced; two now lead with the result
- "Calendar executes all of them" -> "all handlers": the pronoun referred across a sentence boundary after the earlier passive split - "## Call events" -> "## Trigger events": the heading contradicted its own body text, which says "trigger an event manually" the H1 stays a noun phrase on purpose: every page in the section keeps H1 identical to sidebar_label, with title derived from it
[update] grammar and style in Calendar articles
- accessibility updates: modal focus trapping in Grid/Message/Popup, block selection state for screen readers, localizable ARIA labels, larger hit areas for interactive controls - fixes: blockSelection console error in numeric columns, broken aria-labelledby in sub-rows, invalid ARIA attributes and aria-rowindex/aria-rowcount calculations, missing accessible names in inline editors - link to the Grid accessibility guide, matching the v9.3.5 entry
- grid localization: new Accessible strings section with the aria_* keys, their defaults, placeholder rules and a translation example - the same keys added to the default locale snippet - leftover blank line after the removed pro tip cleaned up - whatsnew v9.3.11: update headers in sentence case, localization bullet linked to the guide, block selection wording unified, release date moved to September 14 - whatsnew v9.3.5: update headers and nested labels aligned with the same sentence-case style
- new Accessible names section: table of the aria_* keys for filters, sort triggers, group/row toggles, focus sentinels and drag panel regions, with placeholder rules and a translation example - the same keys added to the default locale snippet - Accessible strings section renamed to Announcements, so the two ARIA sections are told apart by what they cover
- aria_subRow key documented: added to the default locale snippet,
its own table row and a translation example
- Accessible names split into Control names and Sub-row name
subsections, so the sub-row case carries its own table and example
- the {id} placeholder is explained in both subsections, since each
of the two tables uses it
- grid localization: new Validation messages subsection with the aria_value* keys for editor bounds, and the same keys added to the default locale snippet - announcements split into two subsections, sorting/filtering/loading and validation, third column renamed to Announced when - grid accessibility: aria-rowcount and aria-rowindex descriptions corrected to count header rows, per WCAG 1.3.1 - tree mode wording on the summary line aligned with TreeGrid mode used elsewhere in the file
- whatsnew v9.3.11: the accessible names and screen reader announcements bullets link to their localization sections, and the footer line now points to both the accessibility and localization guides - grid accessibility: em-dashes replaced with commas, colons, parentheses or sentence breaks - the two Selection modes headings now put the config name in parentheses and backticks; their anchors change, nothing links to them
- accessibility updates for Colorpicker, Grid, Toolbar, Menu, Sidebar, Ribbon and Window: single tab stop navigation, accessible names and states of controls, and the ARIA dialog pattern for Window - the new ariaLabel option of Toolbar and Window, with the aria_toolbar and aria_dialog locale keys as its fallback - accessible names of Colorpicker, Message, Toolbar and Window moved into the locale system - ten fixes, mostly keyboard focus and ARIA structure in Grid
serhiipylypchuk1991
force-pushed
the
next
branch
from
September 24, 2026 14:33
1be52d4 to
13769ff
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.