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

feat: Make workspace timeline rows obviously clickable #2047

Merged
merged 3 commits into from Jun 7, 2022

Conversation

AbhineetJain
Copy link
Contributor

@AbhineetJain AbhineetJain commented Jun 3, 2022

This PR make workspace timeline rows obviously clickable.

Subtasks

  • added the right arrow
  • update the hover background color
  • apply the same to workspace list
  • remove existing link on the workspace name
  • apply the same to templates list
  • remove existing link on the template name

Fixes #2021
Fixes #2058

Screenshot

Timeline (with hover)

Screen Shot 2022-06-03 at 4 19 19 PM

Templates list

Screen Shot 2022-06-06 at 9 42 56 PM

Workspaces list

Screen Shot 2022-06-06 at 9 42 46 PM

@ammario
Copy link
Member

ammario commented Jun 4, 2022

The grey background looks like it's hovered and not clickable.

I think the Chevron-only is better.

@AbhineetJain
Copy link
Contributor Author

The grey background looks like it's hovered and not clickable.

The grey background is only when it is hovered. The screenshot is from when the row is hovered upon.

@f0ssel
Copy link
Contributor

f0ssel commented Jun 6, 2022

Doesn't have to be this PR, but could we get this treatment on the templates page? @kylecarbs and I just talked about this improvement and think it would help with pushing users to an action on the templates page.

@AbhineetJain AbhineetJain force-pushed the abhineetjain/2021-make-rows-obviously-clickable branch from 7d9db12 to d29238b Compare June 7, 2022 01:48
@AbhineetJain AbhineetJain marked this pull request as ready for review June 7, 2022 02:40
@AbhineetJain AbhineetJain requested a review from a team as a code owner June 7, 2022 02:40
Copy link
Contributor

@presleyp presleyp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work!

@AbhineetJain AbhineetJain force-pushed the abhineetjain/2021-make-rows-obviously-clickable branch from c553981 to 5834755 Compare June 7, 2022 18:51
@AbhineetJain
Copy link
Contributor Author

@presleyp @BrunoQuaresma Can you help with approving the updated UI tests? If you believe I can do that myself, that also works.

@AbhineetJain AbhineetJain enabled auto-merge (squash) June 7, 2022 19:11
@AbhineetJain AbhineetJain merged commit 4b82509 into main Jun 7, 2022
@AbhineetJain AbhineetJain deleted the abhineetjain/2021-make-rows-obviously-clickable branch June 7, 2022 19:11
kylecarbs pushed a commit that referenced this pull request Jun 10, 2022
…2047)

* add right arrow to build table rows

* Add clickable rows to template and workspace list

* Specify 1% width for chevron right
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.

Entire row in workspaces list should be clickable Make workspace timeline rows obviously clickable
4 participants