diff --git a/src/discussions/in-context-topics/TopicPostsView.jsx b/src/discussions/in-context-topics/TopicPostsView.jsx index aa22caae8..55740bb7e 100644 --- a/src/discussions/in-context-topics/TopicPostsView.jsx +++ b/src/discussions/in-context-topics/TopicPostsView.jsx @@ -75,7 +75,7 @@ const TopicPostsView = () => { /> )}
-
handleKeyDown(e)}> +
+
); }; diff --git a/src/discussions/in-context-topics/TopicsView.jsx b/src/discussions/in-context-topics/TopicsView.jsx index e4c02ea97..93320a0be 100644 --- a/src/discussions/in-context-topics/TopicsView.jsx +++ b/src/discussions/in-context-topics/TopicsView.jsx @@ -109,11 +109,11 @@ const TopicsView = () => { {filteredTopics.length === 0 && loadingStatus === RequestStatus.SUCCESSFUL && } )} -
handleKeyDown(e)} > {topicFilter ? ( @@ -126,7 +126,7 @@ const TopicsView = () => { ) : ( )} -
+ ); }; diff --git a/src/discussions/in-context-topics/TopicsView.test.jsx b/src/discussions/in-context-topics/TopicsView.test.jsx index cabc5392d..c74807ebc 100644 --- a/src/discussions/in-context-topics/TopicsView.test.jsx +++ b/src/discussions/in-context-topics/TopicsView.test.jsx @@ -138,12 +138,12 @@ describe('InContext Topics View', () => { it('Section groups should be listed in the middle of the topics list.', async () => { await setupMockResponse(); renderComponent(); - const topicsList = await screen.getByRole('list'); + const topicsList = await screen.getByTestId('list'); const sectionGroups = await screen.getAllByTestId('section-group'); - expect(topicsList.children[1]).toStrictEqual(topicsList.querySelector('.divider')); + expect(topicsList.children[1].querySelector('.divider')).toStrictEqual(topicsList.querySelector('.divider')); expect(sectionGroups.length).toBe(2); - expect(topicsList.children[5]).toStrictEqual(topicsList.querySelector('.divider')); + expect(topicsList.children[3].querySelector('.divider')).toStrictEqual(topicsList.querySelector('.divider')); }); it('A section group should have only a title and required subsections.', async () => { @@ -187,7 +187,7 @@ describe('InContext Topics View', () => { await userEvent.click(subSection); await waitFor(async () => { const backButton = await screen.getByLabelText('Back to topics list'); - const topicsList = await screen.getByRole('list'); + const topicsList = await screen.getByTestId('list'); const subSectionHeading = await screen.findByText(subsectionObject.displayName); const units = await topicsList.querySelectorAll('.discussion-topic'); diff --git a/src/discussions/in-context-topics/topic/ArchivedBaseGroup.jsx b/src/discussions/in-context-topics/topic/ArchivedBaseGroup.jsx index 856f73610..b4da5b51c 100644 --- a/src/discussions/in-context-topics/topic/ArchivedBaseGroup.jsx +++ b/src/discussions/in-context-topics/topic/ArchivedBaseGroup.jsx @@ -23,7 +23,7 @@ const ArchivedBaseGroup = ({ ), [archivedTopics]); return ( - <> +
  • {showDivider && ( <>
    @@ -37,7 +37,7 @@ const ArchivedBaseGroup = ({
    {intl.formatMessage(messages.archivedTopics)}
    {renderArchivedTopics}
    - +
  • ); }; diff --git a/src/discussions/in-context-topics/topic/SectionBaseGroup.jsx b/src/discussions/in-context-topics/topic/SectionBaseGroup.jsx index faca47362..21b8d7000 100644 --- a/src/discussions/in-context-topics/topic/SectionBaseGroup.jsx +++ b/src/discussions/in-context-topics/topic/SectionBaseGroup.jsx @@ -60,7 +60,7 @@ const SectionBaseGroup = ({ ), [section, sectionUrl, isSelected]); return ( -
    )} -
    + ); }; diff --git a/src/discussions/in-context-topics/topic/Topic.jsx b/src/discussions/in-context-topics/topic/Topic.jsx index fb3d0caf9..de491d0ee 100644 --- a/src/discussions/in-context-topics/topic/Topic.jsx +++ b/src/discussions/in-context-topics/topic/Topic.jsx @@ -25,7 +25,7 @@ const Topic = ({ }); return ( - <> +
  • )} - +
  • ); }; diff --git a/src/discussions/learners/LearnersView.jsx b/src/discussions/learners/LearnersView.jsx index f80e52216..d2e7af32a 100644 --- a/src/discussions/learners/LearnersView.jsx +++ b/src/discussions/learners/LearnersView.jsx @@ -76,7 +76,7 @@ const LearnersView = () => { onClear={handleOnClear} /> )} -
    +
    ); }; diff --git a/src/discussions/learners/learner/LearnerCard.jsx b/src/discussions/learners/learner/LearnerCard.jsx index 6ce9cfbdf..7bfecf5bc 100644 --- a/src/discussions/learners/learner/LearnerCard.jsx +++ b/src/discussions/learners/learner/LearnerCard.jsx @@ -21,41 +21,43 @@ const LearnerCard = ({ learner }) => { })(); return ( - -
    + - -
    -
    -
    -
    - {username} +
    + +
    +
    +
    +
    + {username} +
    + {threads !== null && ( + + )}
    - {threads !== null && ( - - )}
    -
    - + + ); }; diff --git a/src/discussions/posts/PostsView.jsx b/src/discussions/posts/PostsView.jsx index b2fa7d208..f97102ff0 100644 --- a/src/discussions/posts/PostsView.jsx +++ b/src/discussions/posts/PostsView.jsx @@ -100,9 +100,9 @@ const PostsView = () => { )}
    -
    handleKeyDown(e)}> +
      handleKeyDown(e)}> {postsListComponent} -
    +
    ); }; diff --git a/src/discussions/posts/post/PostLink.jsx b/src/discussions/posts/post/PostLink.jsx index 63f7659d6..fc9fa818c 100644 --- a/src/discussions/posts/post/PostLink.jsx +++ b/src/discussions/posts/post/PostLink.jsx @@ -57,100 +57,104 @@ const PostLink = ({ ); return ( - -
    + - -
    -
    -
    -
    - + +
    +
    +
    +
    + + {title} + + + {isPostPreviewAvailable(previewBody) + ? previewBody + : intl.formatMessage(messages.postWithoutPreview)} + +
    + {showAnsweredBadge && ( + + {' '}answered + + )} + {canSeeReportedBadge && ( + + {intl.formatMessage(messages.contentReported)} + {' '}reported + + )} + {pinned && ( + )} - > - {title} - - - {isPostPreviewAvailable(previewBody) ? previewBody : intl.formatMessage(messages.postWithoutPreview)} -
    - {showAnsweredBadge && ( - - {' '}answered - - )} - {canSeeReportedBadge && ( - - {intl.formatMessage(messages.contentReported)} - {' '}reported - - )} - {pinned && ( - - )}
    + +
    - -
    -
    - {!showDivider && pinned &&
    } - + {!showDivider && pinned &&
    } + + ); }; diff --git a/src/discussions/topics/TopicsView.jsx b/src/discussions/topics/TopicsView.jsx index 5ea4622d2..55274d9e7 100644 --- a/src/discussions/topics/TopicsView.jsx +++ b/src/discussions/topics/TopicsView.jsx @@ -92,10 +92,10 @@ const TopicsView = () => { onClear={handleOnClear} /> )} -
    handleKeyDown(e)}> +
      handleKeyDown(e)}> -
    + { filteredTopicsCount === 0 && loadingStatus === RequestStatus.SUCCESSFUL diff --git a/src/discussions/topics/topic-group/TopicGroupBase.jsx b/src/discussions/topics/topic-group/TopicGroupBase.jsx index cf6cab52b..7377caf99 100644 --- a/src/discussions/topics/topic-group/TopicGroupBase.jsx +++ b/src/discussions/topics/topic-group/TopicGroupBase.jsx @@ -64,7 +64,7 @@ const TopicGroupBase = ({ } return ( -
    {renderFilteredTopics} -
    + ); };