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

Project: create view UI updates #150

Closed
agjohnson opened this issue Mar 9, 2023 · 3 comments · Fixed by #186
Closed

Project: create view UI updates #150

agjohnson opened this issue Mar 9, 2023 · 3 comments · Fixed by #186
Assignees
Labels
Bug A bug

Comments

@agjohnson
Copy link
Contributor

agjohnson commented Mar 9, 2023

This UI is helpful, but the button is not:

image

There's no link here, we probably want to point to a docs page.

And this can use project chips now, for a popup view of the projects:

image

@agjohnson agjohnson added the Bug A bug label Mar 9, 2023
@agjohnson agjohnson changed the title Project: add view error leads nowhere Project: add view updates Mar 9, 2023
@agjohnson agjohnson changed the title Project: add view updates Project: create view UI updates Mar 9, 2023
@agjohnson agjohnson self-assigned this Mar 29, 2023
@github-project-automation github-project-automation bot moved this to Planned in 📍Roadmap Mar 29, 2023
@agjohnson agjohnson moved this from Planned to In progress in 📍Roadmap Jun 22, 2023
@agjohnson
Copy link
Contributor Author

I wanted to use the project chips on this list, but I can't because the API data returned in the matches field is only id and project url. This could be expanded to include more fields, but then this is still an API driven UI and the chips UI is a Django template. I've made them labels for now so it looks a little better, but it would look much better with image avatars at very least.

@agjohnson
Copy link
Contributor Author

With #181, I was able to use a similar styling on the list of projects to the normal project chips -- it shows the remote repository avatar instead of the FA external link icon. Without #181, the related project list is at least a list of labels though.

@agjohnson
Copy link
Contributor Author

I've opened up readthedocs/readthedocs.org#10513 to address using the V3 API for remote repositories. I need the search param to use the new API and need the new API to get a full project instance response in the remote repo listing search.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant