Quick access to bookmarks and messages on user menu

Using the new profile menu on mobile, I notice that clicking any of these buttons

shows basically the same result - the top of the user profile page, which only has a navigation menu. You have to scroll down a lot to actually see what you came for.

Instead of that, why not show a short list of what you clicked on, right in the menu?

Bookmarks:

Messages:

16 curtidas

I agree it’s odd to arrive at basically the profile menu on mobile when clicking these links. the idea of displaying results in the menu panel, like notifications, is interesting! or at least jump down to an anchor in the results?

2 curtidas

This problem exists for a long time. It makes it quite difficult to use the profile on mobile.

https://meta.discourse.org/t/profile-page-on-mobile-selecting-menu-item-should-jump-to-the-selected-content/19896?u=gerhard

6 curtidas

I like this idea! Although you’d need a “tab” (since that’s basically how these would function now) for notifications as well, to return to it.

Seems like 80+% of the time you’re only looking for recent bookmarks, messages, etc., so this might be useful.

You might also need a divider between the tabs and the settings icon though, since they now have different functions.

2 curtidas

Basically profile page needs fixing. As @sam indicated it might be better if, when you view your own profile only, we compress the top part. It’s especially large on mobile too…

2 curtidas

Mobile looks so much better these days :slight_smile: user page is much improved.

I still sort of like the idea of having a “quick access” somehow, cause despite the improvements I still avoid the user page quite a lot.

2 curtidas

Hello everyone,

I started working on this one. Here is a part of my progress so far.

6 curtidas

I think this change looks good, well done @nbianca, provided “view older {x}…” is updated to be conditional and link to the right place in each view:

  • view older bookmarks…
  • view older messages…
  • view older notifications…

technically assignments are included here as well @sam –

This should be reviewed closely as it is a reasonably large change…

8 curtidas

Isso ainda Ă© pr-welcome? Eu adoraria tentar implementar isso!

Li o tĂłpico e o Ăşltimo PR, mas nĂŁo tenho 100% de certeza sobre:

  • ApĂłs fechar e reabrir o menu do usuário (sem recarregar), ele deve sempre mostrar todas as notificações?
  • Se nĂŁo, quando novas notificações chegarem, estou assumindo que abrir o menu do usuário deve sempre mostrar todas as notificações?
5 curtidas

Acho que gostaria de ver isso primeiro em um componente de tema. Melhorias na API que tornem esse componente de tema viável (e que limpem o núcleo, conforme necessário) são totalmente bem-vindas ao longo do caminho. Mas acho que o tema é o lugar perfeito para testar esse conceito!

Depois, podemos instalar no Meta e “viver” com ele por um tempo para ver se nos agrada. Além disso, nossa API principal e a infraestrutura de componentes de tema ficam melhores ao longo do caminho, o que é uma grande vantagem.

9 curtidas

O componente do tema está em: GitHub - xrav3nz/discourse-quick-access · GitHub. Sugestões são bem-vindas!

Garanti que o design seja fácil para que outros plugins ou temas forneçam seu próprio painel de acesso rápido. Vou documentar isso em detalhes (ou enviar um PR com um exemplo para o discourse-assign) se e quando houver interesse.

Uma rápida demonstração:

Cor do painel selecionado

Por enquanto, ele reutiliza a cor do estado de hover.

Ir para a página completa

  • VocĂŞ ainda pode clicar com o botĂŁo direito ou Ctrl+clicar para abrir a página completa em uma nova aba, ou
  • Clicar no link na parte inferior (ele sempre aparecerá para marcadores e mensagens), ou
  • Clicar duas vezes no mesmo painel de acesso rápido.

Estado vazio

Não tenho certeza sobre a melhor abordagem para isso. Por enquanto, adicionei o texto de ajuda do estado vazio existente de suas respectivas “páginas completas”. O dos marcadores pode ser longo demais para um painel de acesso rápido.

21 curtidas

@xrav3nz isso parece ótimo! Isso já está funcionando? Adicionei o componente e ainda tenho o comportamento antigo.

1 curtida

Trabalho incrível, está muito polido. Adoro que um segundo clique leve à página.

Isso está instalado aqui no Meta, basta selecionar Sam's Simple Theme para testar no menu hambúrguer.

Algumas sugestões de melhoria:

  1. Isso deveria funcionar com o discourse-assign. Tente instalá-lo e atribuir algo a você mesmo.

  1. Sinto que o amarelo está excessivo em termos de estilo. @Johani / @awesomerobot, alguma ideia para um estilo menos chamativo?

  2. Não tenho certeza do que fazer com isso, mas no mobile os alvos de clique são simplesmente pequenos demais. Talvez o componente deveria deixar os elementos um pouco maiores para que seja possível alterá-los?

  3. “Prefs” está “diferente”. Talvez devesse ser o mesmo e adicionar uma lista com Summary (Resumo), Activity (Atividade) ou, talvez, uma lista com Account (Conta), Profile (Perfil), Emails (E-mails) e assim por diante.

    a. Na verdade… pensando melhor, talvez você devesse eliminar o ícone de engrenagem e usar o ícone “pessoa sam” para abrir um painel com Summary, Activity e outros. Isso proporcionaria uma área de clique maior para as abas.

No geral… trabalho espetacular!

13 curtidas

Muito bem @xrav3nz :clap: isso ficou incrĂ­vel!

Podemos seguir o mesmo padrão para os ícones e menus do cabeçalho, algo como isso

e aqui está como ficaria com uma notificação não lida no topo

15 curtidas

Sou eu aqui. Feliz em ver que uma ideia pode persistir por quatro anos e ainda se tornar uma adição útil!

Esse Ă© o poder do Discourse, em poucas palavras.

16 curtidas

Essa funcionalidade é incrível para os favoritos! :heart_eyes: Também adoraria vê-la para atribuição.

Acho que a borda que o @johani montou pode ser um pouco sutil demais, mas eu gostei… então vamos com essa.

14 curtidas

Obrigado pelo feedback! Acabei de enviar uma atualização para abordar os dois primeiros itens. Vou dar seguimento aos outros dois ainda esta semana (#4 exigirá primeiro um PR menor no core).

Maravilhoso e obrigado pelo mockup!

:wink:

Além disso, este commit mostra o quão simples é para outros temas/plugins introduzirem um novo painel de acesso rápido!

12 curtidas

Isso é fantástico! Quão fantástico você pergunta? Tão fantástico que ativei em todos os temas no meta.

Fiquem à vontade para testar isso e dar feedback. Na minha opinião, isso está quase pronto para inclusão no núcleo, mas temos algumas coisas chatas para resolver aqui.

A engrenagem :gear: não é uma aba. Ela parece muito fora de lugar. Se você a remover, poderá deixar todas as abas um pouco mais largas, o que ajudará a ter alvos de clique maiores no mobile! Acredito que você deva tentar removê-la.

Mostrar “Sair” em todas as abas, para mim, parece demais. Por que estamos convidando as pessoas a sair só porque estão olhando as notificações? Parece uma mistura de preocupações.

Minha recomendação aqui é:

  1. Remova “Sair” de todas as subabas.
  2. Adicione uma nova aba ao clicar em pessoa sam à esquerda. Ela pode conter links para “resumo”, “atividade”, “mensagens”, “preferências” e “sair”.

Uma vez que esses detalhes sejam resolvidos, estou pensando que isso Ă© um candidato ideal para inclusĂŁo no nĂşcleo.

21 curtidas

Isso Ă© incrĂ­vel!

Notei um problema muito pequeno: parece que este componente está afetando a cor dos ícones em outros lugares do aplicativo (por exemplo, o status “assistindo”). Normalmente, o ícone é azul aqui:

Gosto dessa ideia. Que tal também adicionar acesso rápido ao tema e ao tamanho da fonte na “aba do sam”? O menu hambúrguer aqui no Meta está ficando bastante poluído com opções de tema, então mover isso para outro lugar seria ótimo.

10 curtidas

Eu realmente gosto disso. Bom trabalho a todos os envolvidos!

Um grande +1 nisso — acho que isso tornará o plugin assign muito mais útil e acionável.

Gosto dessas mudanças também e estou ansioso para experimentá-las. Não tenho certeza se é uma boa ideia esconder o “sair” atrás de outro clique, mas concordo que parece fora de lugar onde está agora, então estou disposto a tentar. Eu ia sugerir colocá-lo onde o ícone de preferências está agora, mas concordo que eles parecem fora de lugar e não queremos incentivar as pessoas a sair sempre que acessarem esse menu.

Seria bom também se fosse possível fornecer um ou dois links personalizados dentro de person sam. Usamos SSO do WordPress, então precisamos enviar as pessoas para o WordPress para editar suas configurações de perfil e e-mail.

6 curtidas