Skip to content

DEV: Small fixes and tweaks in the Glimmer Post Stream #33112

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

megothss
Copy link
Contributor

@megothss megothss commented Jun 6, 2025

  • Fix displaying the time gap
  • Do not display the counter on links with zero posts
  • Adjust button padding and icon margin in the post metadata button indicators (edit)

megothss added 3 commits June 6, 2025 17:44
Update `time-gap` component structure for improved semantics and consistency. Correct the date property reference in `post-stream` from `createdAt` to `created_at` to match the model property.
Ensure links with zero clicks do not render the "data-clicks" attribute when the value is zero.

 Added integration tests to verify correct handling of links with zero and non-zero click counts.
Updated button padding to use default values and removed the right margin for icons within topic posts. These changes ensure consistent spacing and alignment across the UI.
@megothss megothss marked this pull request as ready for review June 10, 2025 02:11
@megothss megothss changed the title DEV: Fix displaying the time gap in the Glimmer Post Stream DEV: Small fixes and tweaks in the Glimmer Post Stream Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants