Skip to content

Commit

Permalink
refactor: use sentence case for interaction (#1558)
Browse files Browse the repository at this point in the history
* refactor: rename `Done` and `Read` to `done` and `read`

Signed-off-by: Adam Setch <[email protected]>

* refactor: mark repository sentence case

Signed-off-by: Adam Setch <[email protected]>

---------

Signed-off-by: Adam Setch <[email protected]>
  • Loading branch information
setchy authored Sep 27, 2024
1 parent 8797b08 commit fca0767
Show file tree
Hide file tree
Showing 9 changed files with 79 additions and 79 deletions.
4 changes: 2 additions & 2 deletions src/components/NotificationRow.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ describe('components/NotificationRow.tsx', () => {
</AppContext.Provider>,
);

fireEvent.click(screen.getByTitle('Mark as Read'));
fireEvent.click(screen.getByTitle('Mark as read'));
expect(markNotificationRead).toHaveBeenCalledTimes(1);
});

Expand All @@ -224,7 +224,7 @@ describe('components/NotificationRow.tsx', () => {
</AppContext.Provider>,
);

fireEvent.click(screen.getByTitle('Mark as Done'));
fireEvent.click(screen.getByTitle('Mark as done'));
expect(markNotificationDone).toHaveBeenCalledTimes(1);
});

Expand Down
4 changes: 2 additions & 2 deletions src/components/NotificationRow.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ export const NotificationRow: FC<INotificationRow> = ({
<HoverGroup>
{isMarkAsDoneFeatureSupported(notification.account) && (
<InteractionButton
title="Mark as Done"
title="Mark as done"
icon={CheckIcon}
size={Size.MEDIUM}
onClick={() => {
Expand All @@ -142,7 +142,7 @@ export const NotificationRow: FC<INotificationRow> = ({
/>
)}
<InteractionButton
title="Mark as Read"
title="Mark as read"
icon={ReadIcon}
size={Size.SMALL}
onClick={() => {
Expand Down
4 changes: 2 additions & 2 deletions src/components/RepositoryNotifications.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ describe('components/RepositoryNotifications.tsx', () => {
</AppContext.Provider>,
);

fireEvent.click(screen.getByTitle('Mark Repository as Read'));
fireEvent.click(screen.getByTitle('Mark repository as read'));

expect(markRepoNotificationsRead).toHaveBeenCalledWith(
mockSingleNotification,
Expand All @@ -80,7 +80,7 @@ describe('components/RepositoryNotifications.tsx', () => {
</AppContext.Provider>,
);

fireEvent.click(screen.getByTitle('Mark Repository as Done'));
fireEvent.click(screen.getByTitle('Mark repository as done'));

expect(markRepoNotificationsDone).toHaveBeenCalledWith(
mockSingleNotification,
Expand Down
4 changes: 2 additions & 2 deletions src/components/RepositoryNotifications.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ export const RepositoryNotifications: FC<IRepositoryNotifications> = ({
<HoverGroup>
{isMarkAsDoneFeatureSupported(repoNotifications[0].account) && (
<InteractionButton
title="Mark Repository as Done"
title="Mark repository as done"
icon={CheckIcon}
size={Size.MEDIUM}
onClick={(event: MouseEvent<HTMLElement>) => {
Expand All @@ -96,7 +96,7 @@ export const RepositoryNotifications: FC<IRepositoryNotifications> = ({
/>
)}
<InteractionButton
title="Mark Repository as Read"
title="Mark repository as read"
icon={ReadIcon}
size={Size.SMALL}
onClick={(event: MouseEvent<HTMLElement>) => {
Expand Down
32 changes: 16 additions & 16 deletions src/components/__snapshots__/AccountNotifications.test.tsx.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

48 changes: 24 additions & 24 deletions src/components/__snapshots__/NotificationRow.test.tsx.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit fca0767

Please sign in to comment.