Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Display the activity views count - EXO-74854 - Meeds-io/MIPs#161 #4129

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

sofyenne
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Oct 22, 2024
@sofyenne sofyenne changed the title feat: Display the the activity views count - EXO-74854 - Meeds-io/MIPs#161 feat: Display the activity views count - EXO-74854 - Meeds-io/MIPs#161 Oct 22, 2024
@sofyenne sofyenne requested a review from azayati October 22, 2024 14:02
@@ -122,6 +122,19 @@
@click="displayFullContent">
<span class="pl-6">{{ $t('UIActivity.label.seeMore') }}</span>
</v-btn>
<div
v-if="activityViews"
:title="activityViewsTooltip"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be v-tooltip instead IMO as it is done in news detail

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To follow the guidelines, we use the same navigator tooltip as the activity link title and summary, as confirmed by the PO!

Copy link
Member

@azayati azayati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@azayati azayati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The label displayed by the tooltip should be updated same as content detail, we need two labels "view" and "views"

Copy link

sonarcloud bot commented Oct 23, 2024

@sofyenne
Copy link
Contributor Author

The label displayed by the tooltip should be updated same as content detail, we need two labels "view" and "views"

done!

Copy link
Member

@azayati azayati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sofyenne sofyenne merged commit 855762f into feature/experience Oct 23, 2024
5 checks passed
@sofyenne sofyenne deleted the Task/74854 branch October 23, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants