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.
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?
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.
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…
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…
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.
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.
Sinto que o amarelo está excessivo em termos de estilo. @Johani / @awesomerobot, alguma ideia para um estilo menos chamativo?
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?
“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.
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).
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.
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.
Adicione uma nova aba ao clicar em pessoa sam à esquerda. Ela pode conter links para “resumo”, “atividade”, “mensagens”, “preferências” e “sair”.