status component: Fix repeater name overflowing #383

Merged
floatingghost merged 1 commit from yukijoou/akkoma-fe:fix-status-usernames into develop 2024-06-25 21:34:16 +00:00
2 changed files with 23 additions and 8 deletions

View file

@ -266,6 +266,16 @@
color: $fallback--cGreen;
color: var(--cGreen, $fallback--cGreen);
}
.right-side {
display: flex;
align-items: center;
gap: 0.3em;
}
.repeat-tooltip {
flex-shrink: 0;
}
}
.repeater-avatar {

View file

@ -83,7 +83,7 @@
:user="statusoid.user"
/>
<div class="right-side faint">
<span
<div
class="status-username repeater-name"
:title="retweeter"
>
@ -100,14 +100,19 @@
v-else
:to="retweeterProfileLink"
>{{ retweeter }}</router-link>
</span>
</div>
{{ ' ' }}
<FAIcon
icon="retweet"
class="repeat-icon"
:title="$t('tool_tip.repeat')"
/>
{{ $t('timeline.repeated') }}
<div
class="repeat-tooltip"
>
<FAIcon
icon="retweet"
class="repeat-icon"
:title="$t('tool_tip.repeat')"
/>
yukijoou marked this conversation as resolved Outdated

This only includes the actual “repeated” text into the flex-shrink: 0; section. Testing the change shows the icon doesn’t get hidden regardless. Do you think it’d make sense to still include the icon into the flex-shrink: 0; section anyway to better communicate the desired output (not hiding the icon)?

This only includes the actual “repeated” _text_ into the `flex-shrink: 0;` section. Testing the change shows the icon doesn’t get hidden regardless. Do you think it’d make sense to still include the icon into the `flex-shrink: 0;` section anyway to better communicate the desired output (not hiding the icon)?

indeed, seems like this would be cleaner!

indeed, seems like this would be cleaner!
{{ $t('timeline.repeated') }}
</div>
</div>
</div>