Skip to content

Commit 23260f0

Browse files
committed
fix: apply PR review feedback
- Use hover:shadow-fg/5 for federated articles shadow - Scope prose link hover to exclude no-underline links - Add group hover for Bluesky icon (blue on card hover) - Revert button-primary hover to original accent style - Use separate class/v-bind for avatar shrink-0 - Fix pt-BR duplicate JSON keys from merge conflict
1 parent a334522 commit 23260f0

File tree

6 files changed

+8
-83
lines changed

6 files changed

+8
-83
lines changed

app/components/BlueskyComment.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ function getHostname(uri: string): string {
5151
:href="`https://bsky.app/profile/${comment.author.handle}`"
5252
target="_blank"
5353
rel="noopener noreferrer"
54-
:class="['shrink-0', depth > 0 ? 'hidden sm:block' : '']"
54+
class="shrink-0"
55+
:class="depth > 0 ? 'hidden sm:block' : ''"
5556
>
5657
<img
5758
v-if="comment.author.avatar"

app/components/Link/Base.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ const keyboardShortcutsEnabled = useKeyboardShortcuts()
101101
'text-xs px-2 py-0.5': isButtonSmall,
102102
'bg-transparent text-fg hover:(bg-fg/10 text-accent) focus-visible:(bg-fg/10 text-accent) aria-[current=true]:(bg-fg/10 text-accent border-fg/20 hover:enabled:(bg-fg/20 text-fg/50))':
103103
variant === 'button-secondary',
104-
'text-bg bg-fg hover:(bg-fg/80 text-bg) focus-visible:(bg-fg/80 text-bg) aria-current:(bg-fg text-bg border-fg hover:enabled:(text-bg/50))':
104+
'text-bg bg-fg hover:(bg-fg/50 text-accent) focus-visible:(bg-fg/50) aria-current:(bg-fg text-bg border-fg hover:enabled:(text-bg/50))':
105105
variant === 'button-primary',
106106
}"
107107
:to="to"

app/components/global/BlogPostFederatedArticles.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ const federatedArticles = computed(() => {
5555
rel="noopener noreferrer"
5656
v-for="article in federatedArticles"
5757
:key="article.url"
58-
class="grid grid-cols-[auto_1fr] gap-x-5 no-underline hover:no-underline rounded-lg border border-border p-4 transition-all hover:shadow-md hover:shadow-black/5 dark:hover:shadow-white/5 hover:border-border-hover"
58+
class="grid grid-cols-[auto_1fr] gap-x-5 no-underline hover:no-underline rounded-lg border border-border p-4 transition-all hover:shadow-md hover:shadow-fg/5 hover:border-border-hover"
5959
>
6060
<AuthorAvatar v-if="article.author" :author="article.author" size="md" class="row-span-2" />
6161
<div class="flex flex-col">

app/components/global/BlogPostWrapper.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ const blueskyPostUri = computed(() => blueskyLink.value?.postUri ?? null)
7878
@apply prose dark:prose-invert;
7979
}
8080
81-
:deep(.prose a:not(.not-prose a)) {
81+
:deep(.prose a:not(.not-prose a):not([class*='no-underline'])) {
8282
text-decoration: underline;
8383
text-underline-offset: 0.2rem;
8484
text-decoration-thickness: 1px;
@@ -88,7 +88,7 @@ const blueskyPostUri = computed(() => blueskyLink.value?.postUri ?? null)
8888
color 0.2s;
8989
}
9090
91-
:deep(.prose a:not(.not-prose a):hover) {
91+
:deep(.prose a:not(.not-prose a):not([class*='no-underline']):hover) {
9292
text-decoration-color: var(--fg);
9393
color: var(--fg);
9494
}

app/components/global/BlueskyPostEmbed.client.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,11 @@ const postUrl = computed(() => {
107107
:href="postUrl ?? '#'"
108108
target="_blank"
109109
rel="noopener noreferrer"
110-
class="not-prose block rounded-lg border border-border bg-bg-subtle p-4 sm:p-5 no-underline hover:border-border-hover transition-colors duration-200 relative"
110+
class="not-prose block rounded-lg border border-border bg-bg-subtle p-4 sm:p-5 no-underline hover:border-border-hover transition-colors duration-200 relative group"
111111
>
112112
<!-- Bluesky icon -->
113113
<span
114-
class="i-simple-icons:bluesky w-5 h-5 text-fg-subtle absolute top-4 end-4 sm:top-5 sm:end-5"
114+
class="i-simple-icons:bluesky w-5 h-5 text-fg-subtle group-hover:text-blue-500 absolute top-4 end-4 sm:top-5 sm:end-5"
115115
aria-hidden="true"
116116
/>
117117

i18n/locales/pt-BR.json

Lines changed: 0 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -1347,81 +1347,5 @@
13471347
"selection": "0 selecionados | 1 selecionado | {count} selecionados",
13481348
"shortcut": "Pressione \"{key}\" para focar nas ações",
13491349
"button_close_aria_label": "Fechar barra de ações"
1350-
},
1351-
"blog": {
1352-
"title": "Blog",
1353-
"meta_description": "Insights e atualizações da comunidade npmx",
1354-
"heading": "blog",
1355-
"draft_banner": "Este é um rascunho não publicado. Pode estar incompleto ou conter imprecisões.",
1356-
"atproto": {
1357-
"comments": "Comentários",
1358-
"could_not_load_comments": "Não foi possível carregar comentários.",
1359-
"like_count": "{count} curtida | {count} curtidas",
1360-
"like_or_reply_on_bluesky": "Curta esta postagem ou adicione seu comentário no Bluesky",
1361-
"likes_on_bluesky": "Curtidas no Bluesky",
1362-
"loading_comments": "Carregando comentários...",
1363-
"no_comments_yet": "Nenhum comentário ainda.",
1364-
"reply_count": "{count} resposta | {count} respostas",
1365-
"reply_on_bluesky": "Responder no Bluesky",
1366-
"repost_count": "{count} repostagem | {count} repostagens",
1367-
"updating": "Atualizando...",
1368-
"view_on_bluesky": "Ver no Bluesky"
1369-
},
1370-
"author": {
1371-
"view_profile": "Veja o perfil de {name} no Bluesky"
1372-
},
1373-
"draft_badge": "Rascunho"
1374-
},
1375-
"error": {
1376-
"401": "Não autorizado",
1377-
"404": "Página não encontrada",
1378-
"500": "Erro do Servidor Interno",
1379-
"503": "Serviço não disponível",
1380-
"default": "Algo deu errado"
1381-
},
1382-
"profile": {
1383-
"description": "Descrição",
1384-
"display_name": "Nome visível",
1385-
"invite": {
1386-
"compose_text": "Olá, {'@'}{handle}! Você já conferiu npmx.dev? É um navegador para o registro npm rápido, moderno e de código aberto.\nhttps://npmx.dev",
1387-
"message": "Parece que eles ainda não estão usando o npmx. Quer contar a eles sobre isso?",
1388-
"share_button": "Compartilhar no Bluesky"
1389-
},
1390-
"likes": "Curtidas",
1391-
"no_description": "Sem descrição",
1392-
"not_found": "Perfil não encontrado",
1393-
"not_found_message": "O perfil de {handle} não foi encontrado.",
1394-
"seo_description": "Perfil npmx de {handle}",
1395-
"seo_title": "{handle} - npmx",
1396-
"website": "Site",
1397-
"website_placeholder": "https://example.com"
1398-
},
1399-
"pds": {
1400-
"community": {
1401-
"description": "Algumas das {count} contas que já estão chamando npmx.social sua casa:",
1402-
"empty": "Nenhum membro da comunidade para exibir.",
1403-
"error": "Falha ao carregar a comunidade PDS.",
1404-
"loading": "Carregando comunidade PDS...",
1405-
"new_accounts": "...e mais {count} que são novos na atmosfera",
1406-
"title": "Quem está aqui",
1407-
"view_profile": "Veja o perfil de {handle}"
1408-
},
1409-
"join": {
1410-
"description": "Esteja você criando sua primeira conta na atmosfera ou migrando uma já existente, você pertence aqui. Você pode migrar sua conta atual sem perder seu controle, suas postagens ou seus seguidores.",
1411-
"migrate": "Migrar com PDS MOOver",
1412-
"title": "Junte-se à comunidade"
1413-
},
1414-
"meta_description": "O AT Protocol Personal Data Server (PDS) oficial para a comunidade npmx.",
1415-
"server": {
1416-
"infrastructure_label": "Infraestrutura:",
1417-
"infrastructure_value": "Hospedado em Hetzner",
1418-
"learn_more": "Aprenda como o npmx usa a atmosfera",
1419-
"location_label": "Localização:",
1420-
"location_value": "Nuremberga, Alemanha",
1421-
"privacy_label": "Privacidade:",
1422-
"privacy_value": "Sujeito às rigorosas leis de proteção de dados da UE",
1423-
"title": "Detalhes do servidor"
1424-
},
1425-
"title": "npmx.social"
14261350
}
14271351
}

0 commit comments

Comments
 (0)