feat: Add summary overlays with named elements (MIN-315)
3 unresolved threads
3 unresolved threads
Closes MIN-315
Objective:
Implement summary overlays with named elements for submap links.
What has been done:
- providing overlay element name
- optional grouping of overlays summaries
- selecting unique overlay summary elements
- unit tests
Merge request reports
Activity
Filter activity
11 12 export const GroupedOverlayAxes = ({ overlay, axes }: Props): JSX.Element | null => { 13 const { idObject, name } = overlay; 14 const overlayAxes = axes.filter(axis => axis.overlayId === idObject); 15 16 if (overlayAxes.length === SIZE_OF_EMPTY_ARRAY) { 17 return null; 18 } 19 20 return ( 21 <div className="flex flex-col gap-2 rounded-lg border border-divide p-4"> 22 <div className="font-bold" data-testid="grouped-overlay-title"> 23 {name} 24 </div> 25 <div className="my-1 h-[1px] w-full bg-divide" /> 26 {getAxesSortedByValue(overlayAxes).map(axis => ( changed this line in version 2 of the diff
8 if (axes.length === ZERO) { 15 export const OverlayData = ({ 16 isShowGroupedOverlays = false, 17 isShowOverlayBioEntityName = false, 18 }: Props = {}): JSX.Element | null => { 19 const axes = useOverlaysAxes({ isShowOverlayBioEntityName }); 20 const uniqueAxes = getUniqueAxes(axes); 21 const openedOverlays = useSelector(overlaysOpenedSelector); 22 23 if (uniqueAxes.length === ZERO) { 9 24 return null; 10 25 } 11 26 27 const overlaysAxesContent = ( 28 <div className="flex flex-col gap-2 rounded-lg border border-divide p-4"> 29 {getAxesSortedByValue(uniqueAxes).map(axis => ( changed this line in version 2 of the diff
reset approvals from @mateusz-winiarczyk by pushing to the branch
enabled an automatic merge when the pipeline for 4f8bc229 succeeds
mentioned in commit d7527586
Please register or sign in to reply