DisclosureTriangleRole model wrong?

[dummdidumm]

In #273 the DisclosureTriangleRole model was adjusted and constraints/attributes were added. If I'm understanding this correctly, that means that a <summary> tag now either needs a open or aria-expanded attribute, but both these AFAIK should be on the parent <details> tag instead. So if I'm understanding this correctly, the DisclosureTriangleRole model should instead be reduced to

const DisclosureTriangleRole: AXObjectModelDefinition = {
  relatedConcepts: [
    {
      module: 'HTML',
      concept: {
        constraints: [
          'scoped to a details element',
        ],
        name: 'summary',
      },
    },
  ],
  type: 'widget',
};

It could also very well be that I'm understanding the semantic meaning of attributes incorrectly, in which case could you briefly explain what it means?

[jessebeach]

Here is the summary html element spec for future reference: https://html.spec.whatwg.org/multipage/interactive-elements.html#the-summary-element

[jessebeach]

It looks like the summary element is mapped to the DisclosureTriangle concept.

[jessebeach]

@dummdidumm your proposal is correct.

[jessebeach]

Fixed in #339

[dummdidumm]

@jessebeach sorry to ping - when can we expect this to get released?

[jessebeach]

I'll try to get a release out on Sunday.

[jessebeach]

@dummdidumm I just published v4.0.0: https://www.npmjs.com/package/axobject-query