Skip to content

Commit

Permalink
Use assembly "displayName" field on View titles (#4819)
Browse files Browse the repository at this point in the history
  • Loading branch information
cmdcolin authored Feb 9, 2025
1 parent 11563e3 commit 1746d7e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion packages/app-core/src/ui/App/ViewContainerTitle.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import EditableTypography from '@jbrowse/core/ui/EditableTypography'
import { getSession } from '@jbrowse/core/util'
import { Tooltip } from '@mui/material'
import { observer } from 'mobx-react'
import { makeStyles } from 'tss-react/mui'
Expand Down Expand Up @@ -29,13 +30,14 @@ const ViewContainerTitle = observer(function ({
view: IBaseViewModel
}) {
const { classes } = useStyles()
const { assemblyManager } = getSession(view)
return (
<Tooltip title="Rename view" arrow>
<EditableTypography
value={
view.displayName ||
// @ts-expect-error
`${view.assemblyNames?.join(',') || 'Untitled view'}${
`${view.assemblyNames?.map(r => assemblyManager.get(r)?.displayName).join(',') || 'Untitled view'}${
view.minimized ? ' (minimized)' : ''
}`
}
Expand Down
2 changes: 1 addition & 1 deletion packages/core/assemblyManager/assembly.ts
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ export default function assemblyFactory(
* #getter
*/
get displayName(): string | undefined {
return self.getConf('displayName')
return self.getConf('displayName') || self.getConf('name') || ''
},
/**
* #method
Expand Down

0 comments on commit 1746d7e

Please sign in to comment.