Admin Dashboard #29
Admin Dashboard #29
Conversation
…low for tempaltes to rread optional fields or default them
…nge the styling a little
… post and event pages for now
| (() => { | ||
| const counters = document.querySelectorAll(".counter"); | ||
| for (const counter of counters) { | ||
| createNewCounter(counter, 0, parseInt(counter.innerText), 4500); |
There was a problem hiding this comment.
This animation is cool but it feels too slow. I want to see this information pretty immediately when I open the dashboard. Functionally the admin tools don't need any fancy animations at all but I do kind of like it for flavor. Reduce to 1 second maybe?
There was a problem hiding this comment.
It would also be nice if the animation didn't replay when switching between the admin views (overview/users on the left)
| tooltipElement.classList.add("show"); | ||
|
|
||
| // Position tooltip near the point | ||
| const rect = this.getBoundingClientRect(); |
There was a problem hiding this comment.
Right now it feels like you need to be very precise when hovering over a dot to view the tooltip. It would be nice to make the dot a little bigger maybe? Or include the entire vertical slice of the chart when considering when to show the mouseover tooltip (and also highlight the slice as you move left and right).
| textAnchor: "middle", | ||
| }, | ||
| axisY: { | ||
| axisTitle: "Revenue ($)", |
There was a problem hiding this comment.
This sideways text is kind of hard to read, not sure the best way of improving it though. Maybe set text-orientation to upright or just sacrifice some horizontal real estate to display it horizontally?
| <input | ||
| _="on click | ||
| {% if user.is_writer %} | ||
| if(confirm('Are you sure you want to promote user to a |
There was a problem hiding this comment.
Clicking "cancel" here still changes the user role.
| {% endblock styles %} | ||
| {% block content %} | ||
| <div id="admin/"> | ||
| <aside id="sidebar"> |
There was a problem hiding this comment.
It would be nice to extend the sidebar to /events and /posts, but that would be dependent on the role I guess? Maybe more trouble than it's worth.
| {% block panel %} | ||
| <div id="admin/dashboard/overview"> | ||
| <div class="widget-container" id="widget-users"> | ||
| <a href="dashboard/users"> |
There was a problem hiding this comment.
This ends up linking to "/admin/dashboard/dashboard/users"
| </div> | ||
| </div> | ||
| <div class="widget-container" id="widget-newsletters"> | ||
| <a> |
There was a problem hiding this comment.
This and the "Events" box both link to nothing but have the "open in new tab icon" still.
|
It would be nice if the dashboard were linked-to somewhere from the main page when the user role is admin. |
Todo
/admin/dashboard/overview/dashboard/users/dashboard/events-> redirects to /events (as of right now)/dashboard/posts-> redirects to /events (as of right now)Future Implementation