Skip to content

Tweaks requested by Ciarán#3009

Merged
kodiakhq[bot] merged 4 commits intomainfrom
fix/ui-tweaks-ciaran
Jun 9, 2022
Merged

Tweaks requested by Ciarán#3009
kodiakhq[bot] merged 4 commits intomainfrom
fix/ui-tweaks-ciaran

Conversation

@leog
Copy link
Contributor

@leog leog commented Jun 7, 2022

What does this PR do?

Some UI tweaks @Jaibles asked and were due to be applied

Tweaks for App page

CleanShot.2022-06-01.at.08.44.28.mp4
  • Pending validation for background color white for icons that fill almost the entire room as shown below:

image

Tweaks for Installed Apps selectors

Rounded corners 2px (sm)
image

Copy added for list of calendars connected

image

Availability page mobile view now shows recurring info

Tennis_class___Pro_Example___Cal_com

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@vercel
Copy link

vercel bot commented Jun 7, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Jun 9, 2022 at 8:43PM (UTC)
3 Ignored Deployments
Name Status Preview Updated
docs ⬜️ Ignored (Inspect) Jun 9, 2022 at 8:43PM (UTC)
swagger ⬜️ Ignored (Inspect) Jun 9, 2022 at 8:43PM (UTC)
ui ⬜️ Ignored (Inspect) Jun 9, 2022 at 8:43PM (UTC)

@leog leog added the ♻️ autoupdate tells kodiak to keep this branch up-to-date label Jun 7, 2022
@PeerRich
Copy link
Member

PeerRich commented Jun 9, 2022

those are great, when will this go out of draft @leog ?

@leog
Copy link
Contributor Author

leog commented Jun 9, 2022

those are great, when will this go out of draft @leog ?

I just need to confirm with @Jaibles (DMed him) if we still want the white background color for each app logo in their own page, as they can have a full logo and the background looks weird in those cases.

@ciaranha
Copy link
Member

ciaranha commented Jun 9, 2022

Proceed without white background. Otherwise looks great thanks @leog

@leog leog added the automerge label Jun 9, 2022
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Jun 9, 2022

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.

@leog leog marked this pull request as ready for review June 9, 2022 20:36
@leog leog added the automerge label Jun 9, 2022
@kodiakhq kodiakhq bot merged commit 8cd4201 into main Jun 9, 2022
@kodiakhq kodiakhq bot deleted the fix/ui-tweaks-ciaran branch June 9, 2022 20:49
@zomars zomars mentioned this pull request Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants