-
Notifications
You must be signed in to change notification settings - Fork 26
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
Project taxonomy #1846
Project taxonomy #1846
Conversation
5d787f8
to
fc159ee
Compare
ce1f877
to
9f8994e
Compare
9f8994e
to
25ba39f
Compare
@goose-life @longhotsummer Happy for you to check this out. |
indigo_app/templates/indigo_app/tasks/taxonomy_project_detail.html
Outdated
Show resolved
Hide resolved
indigo_app/templates/indigo_app/tasks/taxonomy_project_detail.html
Outdated
Show resolved
Hide resolved
indigo_app/templates/indigo_app/tasks/taxonomy_project_list.html
Outdated
Show resolved
Hide resolved
f13ad9e
to
03237a7
Compare
03237a7
to
b5c428b
Compare
indigo_app/templates/indigo_app/tasks/taxonomy_task_detail.html
Outdated
Show resolved
Hide resolved
indigo_app/templates/indigo_app/tasks/taxonomy_task_detail.html
Outdated
Show resolved
Hide resolved
looks great overall, I'd love to see a new video once that's possible, and I think some minor tweaks around 'project/s' vs 'topic/s' (though I stand to be corrected on those) |
@@ -24,6 +24,12 @@ | |||
<span class="badge badge-light">{{ tab_count }}</span> | |||
{% endif %} | |||
</a> | |||
<a class="nav-item nav-link {% if view.tab == 'projects' %}active{% endif %}" href="{% url 'taxonomy_task_list' %}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<a class="nav-item nav-link {% if view.tab == 'projects' %}active{% endif %}" href="{% url 'taxonomy_task_list' %}"> | |
<a class="nav-item nav-link {% if view.tab == 'topics' %}active{% endif %}" href="{% url 'taxonomy_task_list' %}"> |
@goose-life I've made a quick loom showing the small changes, its just the filters and the tab label that have changed from the previous loom. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
Enhancements
https://www.loom.com/share/555c8e78802149e9bb66f6794c768871
closes #1450