Skip to content

[FIX] website_membership_group: Set the height for member images so that it stays consistent#143

Merged
ByteMeAsap merged 1 commit into
18.0from
18.0-upd-website_membership_group
May 15, 2026
Merged

[FIX] website_membership_group: Set the height for member images so that it stays consistent#143
ByteMeAsap merged 1 commit into
18.0from
18.0-upd-website_membership_group

Conversation

@ByteMeAsap
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 15, 2026 09:01
@ByteMeAsap ByteMeAsap merged commit fdd5b66 into 18.0 May 15, 2026
1 of 3 checks passed
@ByteMeAsap ByteMeAsap deleted the 18.0-upd-website_membership_group branch May 15, 2026 09:01
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Sets a fixed height on member avatar images in the membership group template so member cards display consistently, plus minor formatting/whitespace cleanups.

Changes:

  • Adds style="height:128px" to the member image element to enforce a consistent image height.
  • Reformats a few template lines (indentation/line wraps) in website.xml.
  • Whitespace/comment tweaks in membership_group_options.esm.js and membership_activity_views.xml.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
website_membership_group/templates/website.xml Adds inline height style on member image; minor reformatting
website_membership_group/static/src/js/membership_group_options.esm.js Adds eslint-disable comment for MutationObserver
membership_activity_project_parent/views/membership_activity_views.xml Trailing newline adjustment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants