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

[C-3651] Add box-shadow to text-input #7259

Merged
merged 2 commits into from
Jan 20, 2024
Merged

Conversation

dylanjeffers
Copy link
Contributor

@dylanjeffers dylanjeffers commented Jan 20, 2024

Description

Adds box-shadow to harmony and harmony-native text-inputs to dramatically improve readability and align with figma

@audius/harmony:
image

@audius/harmony-native:
Sign Up For Audius

Copy link

vercel bot commented Jan 20, 2024

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

1 Ignored Deployment
Name Status Preview Updated (UTC)
audius-web-ssr ⬜️ Ignored (Inspect) Jan 20, 2024 0:38am

@dylanjeffers dylanjeffers requested a review from DejayJD January 20, 2024 00:38
ph='l'
gap={isSmall ? 's' : 'm'}
style={animatedRootStyles}
<InsetShadow
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, I assume this works pretty much the same way as web?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah its not a perfect match, since web is a combination of shadows, but its pretty decent

Copy link
Contributor

Choose a reason for hiding this comment

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

ah yeah that makes sense

Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/react-native-inset-shadow@1.0.3 Transitive: environment, eval, filesystem, network, shell, unsafe +404 176 MB rohailaltaf

View full report↗︎

@audius-infra
Copy link
Collaborator

It looks like there may be some changes to native mobile code, which requires triggering a full app release. Please follow the instructions here: https://www.notion.so/audiusproject/When-to-bump-app-version-2644a8f772364a4d91f44abcba44ce0b?pvs=4. cc @nicoback2 @sliptype

@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/dj-c-3651-inset-shadows

@dylanjeffers dylanjeffers merged commit df201c4 into main Jan 20, 2024
20 of 24 checks passed
@dylanjeffers dylanjeffers deleted the dj-c-3651-inset-shadows branch January 20, 2024 03:33
audius-infra pushed a commit that referenced this pull request Jan 20, 2024
[df201c4] [C-3651] Add box-shadow to text-input (#7259) Dylan Jeffers
[3436391] Remove track player patch (#7256) Dylan Jeffers
[2dd727a] [C-3633] Fix upload tags (#7246) Dylan Jeffers
[336abe5] [PAY-2362] Allow offline download (#7253) Saliou Diallo
[bdc4997] Re-add download buttons to mobile track page (#7254) Reed
[5f2045c] Add auto upgrade logs to vector (#7251) Isaac Solo
[d87e13f] Update styles to wrap properly (#7248) Saliou Diallo
[1599367] [PROTO-1598] Use tRPC in ddex (#7150) Theo Ilie
[09b5dc7] Qm sync (#7250) Steve Perkins
[cc892ea] Remove scheduled release filter from sitemaps (#7247) Isaac Solo
[3211ca2] Fix stylelint (#7245) Saliou Diallo
[270f107] Increase chat reaction emoji sizes (#7241) Reed
[26b0f0c] [PAY-2331] Add lossless download info section in upload (#7219) Saliou Diallo
[2a15a64] v1.5.62 (#7243) Dylan Jeffers
[b94153b] Convert mobile DownloadRow to harmony (#7237) Reed
[62b8611] [C-3597] Add ellipses to track tile titles (#7234) Dylan Jeffers
[eba9299] [C-3678, C-3657] Fix android radial-gradient bug, Fix sign-up panel top border-radius (#7236) Dylan Jeffers
[bf9d2ea] Fix sign ups failing halfway through saga (#7240) JD Francis
[f355286] Re-add downloads when lossless downloads flag disabled (#7235) Reed
[0ffd417] [C-3677] Fix onclick for upload a track chip (#7232) Andrew Mendelsohn
[f9132ca] Filter sitemaps properly (#7233) Raymond Jacobson
[5f8d5d2] Fix max width issue on select artists page (#7229) JD Francis
[dc4fa10] [DOCS] - repair reference links and set config to throw errors on breaks (#7231) Sam Gutentag
[f6ad76f] [PAY-2361] Fix search es with new gated fields (#7230) Saliou Diallo
[5d97832] [PAY-2319] Add offramp on web (#7153) Randy Schott
[eab69ec] [PAY-2364] Fix nft gated tracks (#7228) Saliou Diallo
[12af10e] [C-3637] Add OTP flow for new sign-up (#7227) Dylan Jeffers
[4d68c2a] [C-3642] Handle "user past account creation" edge case scenarios in web (#7225) JD Francis
[60b843d] Fix DDEX dockerfile (#7226) Michelle Brier
[c3c2148] [Web] [Libs] Disconnect dashboard wallet - web and SDK changes (#7209) nicoback2
[b7d03e3] [C-3640] Reduce toast duration app-wide (#7224) Andrew Mendelsohn
[120c8f6] [C-3639] Fix empty album tab state (#7222) Andrew Mendelsohn
[059312d] [C-3621] Fix scrolling issues on new sign up Password & Handle pages (web) (#7204) JD Francis
[557fd96] Get public IP in nginx (#7223) Isaac Solo
[0046ab4] [C-3149 C-3122 C-3051 C-3625 C-3481] Fix add track to playlist (#7221) Dylan Jeffers
[ba711b6] [C-3583 C-3634 C-3404] Add harmony-native to native sign up (#7210) Dylan Jeffers
[335cfa4] Remove yield* in JS file (#7134) Marcus Pasell
[fa7a9d9] Make FilterButton a controlled component (#6993) Marcus Pasell
[1eb3aa1] [C-3649] Update create password screen to move when keyboard opens to display validation (#7220) Kyle Shanks
[55b5192] [PAY-2356] Mobile track screen stems & download section (#7207) Reed
[c0b2b66] [Web] Add OTP to OAuth login flow [C-3643] (#7212) nicoback2
[f5cee04] Openresty container (#7214) Steve Perkins
[33f7597] Export plain button from harmony native (#7217) Kyle Shanks
[13412f3] CircleCI self-hosted runners: lower cleanup thresholds (#7216) Danny
[c57618e] Fix otp after #7196 (#7215) Raymond Jacobson
[31629e6] [C-3638] Fix saga ci (#7211) Dylan Jeffers
[ab09783] Fix ISO date format for dayjs (#7206) Isaac Solo
[abc71bf] Fix stems & downloads appearing erroneously web (#7202) Reed
[fb38737] [PAY-2351] Fix height on add funds (#7172) Raymond Jacobson
[38221f2] [C-3614, C-3615] Update image storage for new sign up flow to conform with old structure (#7200) Kyle Shanks
[b0bf87d] use slog-gorm package (#7199) Steve Perkins
[ce50f32] Gitignore snippets (#7201) Reed
[204894c] Fix audius-cmd test (#7208) Saliou Diallo
[f5226c7] [C-3630] Refactor signon screens to use native-harmony (#7192) Dylan Jeffers
[bcc180f] Bypass OTP for testflight account (#7196) Saliou Diallo
[94ec3d0] Update mobile version to 1.1.83 for release (#7203) Dylan Jeffers
[ab1d3e9] Add USDC gated downloads flow (#6899) Saliou Diallo
[966e438] [C-3608] Increase pressable area for cover photo upload (native) (#7165) JD Francis
[9012df5] [C-2190] Fix track player interruptions (#7198) Dylan Jeffers
[d12d824] [C-3632] Return null instead of NOT_FOUND in trpc-server (#7184) Andrew Mendelsohn
[b391ada] Bump version to 0.6.19 audius-infra
[ecbd9b1] Bump version to 0.6.18 audius-infra
[625839b] Fix repair signup (#7197) Marcus Pasell
[fc5131e] Add sendgrid param (#7195) Raymond Jacobson
[d530b0e] Bump app builds (#7194) Raymond Jacobson
[8d031a4] [PROTO-1599] Add OTP support (#7193) Raymond Jacobson
schottra added a commit that referenced this pull request Jan 22, 2024
…xing

* origin/main: (37 commits)
  [DVRL-3] optimize image assets (#7238)
  PROTO-1607: healthz rm GA dependency (#7264)
  [C-3696] Fix lineups and recommendations (#7263)
  Revert "[C-3510] Move mobile env from system config to ts env (#6935)" (#7268)
  Make ddex tRPC Dockerfile run (#7261)
  Attempt to circumvent docker-compose regression (#7266)
  Bump version to 0.6.20
  qm sync: handle non-200 status, more tests. (#7265)
  [PAY-2367] Add migration to repair tips (#7252)
  [C-3510] Move mobile env from system config to ts env (#6935)
  v1.5.63
  [C-3651] Add box-shadow to text-input (#7259)
  Remove track player patch (#7256)
  [C-3633] Fix upload tags (#7246)
  [PAY-2362] Allow offline download (#7253)
  Re-add download buttons to mobile track page (#7254)
  Add auto upgrade logs to vector (#7251)
  Update styles to wrap properly (#7248)
  [PROTO-1598] Use tRPC in ddex (#7150)
  Qm sync (#7250)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants