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.

Sign up for GitHub

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

Jump to bottom

Fix menu UI/UX issues #198

Merged
merged 1 commit into from Nov 10, 2022
Merged

Fix menu UI/UX issues #198

merged 1 commit into from Nov 10, 2022

Conversation

mariobalca
Copy link
Contributor

@mariobalca mariobalca commented Nov 10, 2022 •
edited

Changes proposed ✍️

  • Tooltips now fade away after 50ms
  • Workspace popup/dropdown now closes as expected

Checklist ✅

  • Label appropriately with type:feature 🚀, type:enhancement ✨, type:bug 🐞, or type:documentation 📜.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated
    • Front-end: frontend/.env.dist
    • Backend: backend/.env.dist, backend/.env.dist.staging, backend/.env.dist.staging.
    • Configuration docs have been updated.
    • Team members only: update environment variables in Password manager and update the team
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@netlify
Copy link

netlify bot commented Nov 10, 2022 •
edited

❌ Deploy Preview for open-crowd-prod failed.

Name Link
🔨 Latest commit bd77e16
🔍 Latest deploy log https://app.netlify.com/sites/open-crowd-prod/deploys/636d0fe78efa3a000ab0809b

@mariobalca mariobalca self-assigned this Nov 10, 2022
@netlify
Copy link

netlify bot commented Nov 10, 2022 •
edited

❌ Deploy Preview for open-devfounders failed.

Name Link
🔨 Latest commit bd77e16
🔍 Latest deploy log https://app.netlify.com/sites/open-devfounders/deploys/636d0fe74291d000092696c4

@joanagmaia
Copy link
Contributor

LGTM 🚀

@joanagmaia joanagmaia merged commit 407886d into main Nov 10, 2022
@joanagmaia joanagmaia deleted the hotfix/menu-tweaks branch November 10, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joanagmaia joanagmaia joanagmaia approved these changes

Assignees

@mariobalca mariobalca

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mariobalca @joanagmaia