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

initial admin comments list #72

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

initial admin comments list #72

wants to merge 6 commits into from

Conversation

aszenz
Copy link
Contributor

@aszenz aszenz commented May 4, 2021

@aszenz
Copy link
Contributor Author

aszenz commented Jun 2, 2021

An update on this:
Right now list of comments are shown unstyled, we will need a card component, i'm looking at how tailwind utility classes work which are used in the project for styling.
Elm now has native tailwind support via https://package.elm-lang.org/packages/matheus23/elm-default-tailwind-modules/2.0.1/, so instead of using postcss we can directly use it too.

@aszenz
Copy link
Contributor Author

aszenz commented Jun 7, 2021

Another update:

This required a bunch of changes to show the comment date in the user's local timezone, specifically the app model structure was changed so that user's time zone information could be retrieved at app init.

Here are some screenshots, the post name is still unimplemented since that requires back end support

Screenshot from 2021-06-08 01-38-28
Screenshot from 2021-06-08 01-37-31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant