- Issue created by @alok_singh
- Merge request !26Issue #3494458: Read more and Add new comment needs some space and style. โ (Open) created by Unnamed author
@sandip poddar, the color is not matching with the theme. It should be "#41449f"
- ๐ฎ๐ณIndia sandip
@tanuja bohra, but in this theme the buttons of comment section is green colour thats why i match the colour of read more and add new comment button with it.
- ๐ฎ๐ณIndia rahuly072
I have tested this MR, and it has been applied successfully. The spacing and style issues with the "Read More" and "Add Comment" buttons have been resolved. I have attached a screenshot for your reference. Thank you!
- ๐ท๐ธSerbia majmunbog
The approach to add specific styling to all article lists is no good.
This should be fixed/improved with a utility class. - ๐ฎ๐ณIndia sandip
Hi @majmunbog,
Thank you for your suggestion! I have implemented the utility class as you recommended to improve the styling approach. Could you please review the changes and share your feedback? Let me know if there are any further refinements needed.
- Issue was unassigned.
- Status changed to Needs work
25 days ago 5:58am 1 April 2025 Hello @sandip poddar,
Thank you for your contribution! I have applied your merge request locally and thoroughly reviewed both the UI and code implementation. While testing, I encountered an issue.
Steps to Reproduce:
- Create some nodes of the Article content type.
- Create a Views page or block (I created a page) displaying Article content.
- Add a comment to any node.
- Navigate to the Views page/blockโan extra empty button is rendering unexpectedly.
I am attaching a screenshot for reference. Could you please take a look into this? Let me know if you need any further details.
Thanks!
- ๐ฎ๐ณIndia sandip
Hi @utkarsh_kumar_singh, Thanks for the review but it seems a different issue and this issue only belongs to styling of the Read more and Add New Comment button only.
For that extra button it does not belong to this issue. Can you please reverify it once. Hi @sandip-poddar,
Thank you for your response! I reverified the issue, and it seems that the extra empty button is indeed being rendered due to the presence of a "hidden" class. However, the styling changes introduced in your code override the display property of this class, causing the button to become visible.
Could you please take a look and confirm if this can be adjusted within the scope of your changes? Let me know your thoughts.
Thanks again for your efforts!
Hi @sandip-poddar,
Thank you for your response! I reviewed the MR locally, and it looks good to me. So I am moving the issue status to RTBC.
Thank you once again.