Skip to content

Commit

Permalink
refactor(location-field): Add comments about id assumptions
Browse files Browse the repository at this point in the history
  • Loading branch information
binh-dam-ibigroup committed Jul 11, 2023
1 parent d3590ae commit e6c2aa7
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions packages/location-field/src/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -195,10 +195,8 @@ const LocationField = ({
return location && !hideExistingValue ? label : "";
};

// FIXME: This class name seems only used in OTP-RR for Percy tests to identify the component.
const formControlClassname = `${locationType}-form-control`;

// We assume there is only up to one location field component for each value in LocationType.
const listBoxId = `${locationType}-listbox`;

const intl = useIntl();
Expand Down Expand Up @@ -939,8 +937,6 @@ const LocationField = ({
value={statusMessages}
/>
</S.HiddenContent>
{/* The results list is always in the DOM (but hidden in desktop if no results are shown),
so that assistive technology can associate the list with the button and input controls. */}
<ItemList
// Hide the list from screen readers if no enabled options are shown.
aria-hidden={hasNoEnabledOptions}
Expand Down

0 comments on commit e6c2aa7

Please sign in to comment.