Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problems with documentation of options accessibleName and helpText. #1661

Open
pixelzoom opened this issue Oct 14, 2024 · 5 comments
Open

Problems with documentation of options accessibleName and helpText. #1661

pixelzoom opened this issue Oct 14, 2024 · 5 comments

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Oct 14, 2024

For phetsims/models-of-the-hydrogen-atom#67 ...

I attempted to learn about accessibleName and helpText from the documentation for those options in ParallelDOM.ts:

  accessibleName?: PDOMValueType | null; // Sets the name of this node, read when this node receives focus and inserted appropriately based on accessibleNameBehavior
  accessibleNameBehavior?: PDOMBehaviorFunction; // Sets the way in which accessibleName will be set for the Node, see DEFAULT_ACCESSIBLE_NAME_BEHAVIOR for example
  helpText?: PDOMValueType | null; // Sets the descriptive content for this node, read by the virtual cursor, inserted into DOM appropriately based on helpTextBehavior
  helpTextBehavior?: PDOMBehaviorFunction; // Sets the way in which help text will be set for the Node, see DEFAULT_HELP_TEXT_BEHAVIOR for example

Problems:

  • Documentation refers to "node" and "Node", when this is the parent class of Node.
  • DEFAULT_ACCESSIBLE_NAME_BEHAVIOR does not exist in the PhET code base.
  • DEFAULT_HELP_TEXT_BEHAVIOR does not exist in the PhET code base.

If those problems were fixed, I'd also say that this documentation is inadequate to get me up to speed on these options. It would be nice if they pointed the reader to more details. But no such detailed documentation currently exists... which will be the topic of another GitHub issue.

Assigning to @zepumph based on what git history tells me.

@kathy-phet
Copy link

Reassigning this issue to @jessegreenberg, as this is aligned with Jesse's work at the moment.

@jessegreenberg @marlitas - Shall we start a Monday.com triage board around dev:description issues in common code repos, so we can triage and progress?

@kathy-phet kathy-phet assigned jessegreenberg and unassigned zepumph Oct 16, 2024
@marlitas
Copy link
Contributor

@kathy-phet yes I think scheduling triage for description is great. Do we want this board to be separate from the alt-input board? I see some issues in the alt-input board related to description which seems to indicate overlap between the two.

@jessegreenberg
Copy link
Contributor

Shall we start a Monday.com triage board around dev:description issues in common code repos, so we can triage and progress

Yes please! The alt input board has been helpful. I think it will be separate enough to have its own board. Some items that are on the alt-input board could move to a description board.

I'll take a look at the documentation here.

@marlitas
Copy link
Contributor

marlitas commented Oct 17, 2024

Great! I'll get one going today and find a time for us to meet. I'm assuming the following people should be involved in the meeting:
JG, JO, TS, BF, KP, (EM if available)

@kathy-phet
Copy link

@marlitas - @pixelzoom will be adding recommendations based on review and @jessegreenberg will also likely add issues from the prototyping work around translation this iteration, so we won't set up a meeting until those issues are in. My current thinking is meet end of this iteration or early next iteration to scope out the list and set priorities for next iteration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants