Skip to content

Commit

Permalink
Clarify comments
Browse files Browse the repository at this point in the history
  • Loading branch information
rayane-d committed Sep 6, 2024
1 parent 71489bb commit 6bd40e0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/components/SelectionListWithModal/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@ function SelectionListWithModal<TItem extends ListItem>(
const [isModalVisible, setIsModalVisible] = useState(false);
const [longPressedItem, setLongPressedItem] = useState<TItem | null>(null);
const {translate} = useLocalize();
// We need to use isSmallScreenWidth instead of shouldUseNarrowLayout https://github.com/Expensify/App/issues/48675
// Because there is a race condition somewhere in the app that causes shouldUseNarrowLayout to change indefinitely in this component
// We need to use isSmallScreenWidth instead of shouldUseNarrowLayout here because there is a race condition that causes shouldUseNarrowLayout to change indefinitely in this component
// See https://github.com/Expensify/App/issues/48675 for more details
const {isSmallScreenWidth} = useResponsiveLayout();
const {selectionMode} = useMobileSelectionMode(true);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,8 @@ function ReportFieldsListValuesPage({
}: ReportFieldsListValuesPageProps) {
const styles = useThemeStyles();
const {translate} = useLocalize();
// We need to use isSmallScreenWidth instead of shouldUseNarrowLayout to use selection mode on small screens only https://github.com/Expensify/App/issues/48724
// We need to use isSmallScreenWidth instead of shouldUseNarrowLayout here to use the mobile selection mode on small screens only
// See https://github.com/Expensify/App/issues/48724 for more details
const {isSmallScreenWidth} = useResponsiveLayout();
const [formDraft] = useOnyx(ONYXKEYS.FORMS.WORKSPACE_REPORT_FIELDS_FORM_DRAFT);
const {selectionMode} = useMobileSelectionMode();
Expand Down

0 comments on commit 6bd40e0

Please sign in to comment.