Skip to content

Commit 5ebb07d

Browse files
authored
Merge pull request #1301 from mathjax/update/semantic-info
Make 'Semantic Info' menu active (not sure why it was disabled)
2 parents ba96ff0 + f5a9b7f commit 5ebb07d

3 files changed

Lines changed: 12 additions & 13 deletions

File tree

ts/a11y/explorer/ExplorerPool.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,8 +170,8 @@ const allExplorers: { [options: string]: ExplorerInit } = {
170170
pool,
171171
doc.explorerRegions.tooltip3,
172172
node,
173-
(x: HTMLElement) => x.hasAttribute('data-semantic-prefix'),
174-
(x: HTMLElement) => x.getAttribute('data-semantic-prefix')
173+
(x: HTMLElement) => x.hasAttribute?.('data-semantic-prefix-none'),
174+
(x: HTMLElement) => x.getAttribute?.('data-semantic-prefix-none')
175175
),
176176
flame: (
177177
doc: ExplorerMathDocument,

ts/a11y/explorer/Region.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -323,14 +323,18 @@ export class ToolTip extends StringRegion {
323323
height: 'auto',
324324
opacity: 1,
325325
'text-align': 'center',
326-
'border-radius': '6px',
326+
'border-radius': '4px',
327327
padding: 0,
328328
'border-bottom': '1px dotted black',
329329
position: 'absolute',
330330
display: 'inline-block',
331331
'background-color': 'white',
332332
'z-index': 202,
333333
},
334+
['.' + ToolTip.className + ' > div']: {
335+
'border-radius': 'inherit',
336+
padding: '0 2px',
337+
},
334338
});
335339
}
336340

ts/ui/menu/Menu.ts

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -823,16 +823,11 @@ export class Menu {
823823
['500%'],
824824
]),
825825
]),
826-
this.submenu(
827-
'Semantic Info',
828-
'Semantic Info',
829-
[
830-
this.checkbox('Type', 'Type', 'infoType'),
831-
this.checkbox('Role', 'Role', 'infoRole'),
832-
this.checkbox('Prefix', 'Prefix', 'infoPrefix'),
833-
],
834-
true
835-
),
826+
this.submenu('Semantic Info', 'Semantic Info', [
827+
this.checkbox('Type', 'Type', 'infoType'),
828+
this.checkbox('Role', 'Role', 'infoRole'),
829+
this.checkbox('Prefix', 'Prefix', 'infoPrefix'),
830+
]),
836831
this.rule(),
837832
this.submenu('Role Description', 'Describe math as', [
838833
this.radioGroup('roleDescription', [

0 commit comments

Comments
 (0)