Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
Fix regression around FacePile with overflow (#11527)
Browse files Browse the repository at this point in the history
* Work around compound-web AvatarStack not applying overlap to non-Avatars

* Fix FacePile overflow tile not being layed out correctly
  • Loading branch information
t3chguy authored Sep 5, 2023
1 parent 9eda619 commit 93ccccc
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions res/css/views/elements/_FacePile.pcss
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ limitations under the License.
width: 30px;
height: 30px;
background-color: $spacePanel-bg-color;
display: inline-block;

&::before {
content: "";
Expand Down
3 changes: 2 additions & 1 deletion src/components/views/elements/FacePile.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,8 @@ const FacePile: FC<IProps> = ({

const pileContents = (
<>
{overflow ? <span className="mx_FacePile_more" /> : null}
{/* XXX: The margin-left is a workaround for Compound's styling excluding this element and being overly specific */}
{overflow ? <span className="mx_FacePile_more" style={{ marginLeft: `calc(${size} * -0.2)` }} /> : null}
{faces}
</>
);
Expand Down

0 comments on commit 93ccccc

Please sign in to comment.