Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update UI #150

Open
1 of 10 tasks
SimonLab opened this issue Jul 24, 2023 · 1 comment
Open
1 of 10 tasks

Update UI #150

SimonLab opened this issue Jul 24, 2023 · 1 comment
Labels
enhancement New feature or enhancement of existing functionality

Comments

@SimonLab
Copy link
Member

SimonLab commented Jul 24, 2023

image

  • "People in chat" doesn't display the names display names on refresh. Make sure Presence is working properly
    image

  • Centre horizontally the chat

  • Make the times more readable human friendly, ex: "12mn ago"

  • Better display the errors linked to the inputs:
    image

  • Add white space top, left and right for the header section to make a bit less sticky to the screen

  • Center "login" and profile image; Make profile image round? Capitalise "login"
    image

  • Remove login button at the button of the page as it already exists on the header

  • Add white space for footer

  • Use textarea instead of input for message value

  • Focus message input when the page load

@SimonLab SimonLab added the enhancement New feature or enhancement of existing functionality label Jul 24, 2023
@SimonLab SimonLab mentioned this issue Jul 26, 2023
@SimonLab
Copy link
Member Author

The presence information is not "liveview" updated because the code is in the root layout:

<div class="text-white width-[10%] float-left ml-3 -mt-5 align-middle">
<b>People in Chat:</b>
<ul>
<%= for name <- @presence do %>
<li>
<%= name %>
</li>
<% end %>
</ul>
</div>

And from https://hexdocs.pm/phoenix_live_view/live-layouts.html

The "root" layout is rendered only on the initial

We need to review how the liveview layout are defined with Phoenix 1.7 and we might need to move this html code in another layout which can display liveview values

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement of existing functionality
Projects
None yet
Development

No branches or pull requests

1 participant