Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

Update styles and semantics for listed for per feedback #660

Merged
merged 5 commits into from
Jan 10, 2022

Conversation

TatisLois
Copy link
Contributor

@TatisLois TatisLois commented Jan 6, 2022

Related to this linear issue
299

Why

Feedback from design on copy and UI

How

  • update tailwind styles for light + dark
  • updated element
  • Spotlight checks if the item is free or a single edition, removes royalties info and updates it's copy
  • Remove star from title of tokencard, checks if the item is free or single and updates copy
  • TokenDetailBody updates copy

Test Plan

Test locally as the listing owner and as an anon account (see buy output)

Screenshots

Screen Shot 2022-01-06 at 11 47 19 PM

Screen Shot 2022-01-06 at 11 47 46 PM

Screen Shot 2022-01-06 at 11 47 40 PM

@vercel
Copy link

vercel bot commented Jan 6, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/showtime/showtime/5cNfu1umSvctXcDPVapj7u3jhdQi
✅ Preview: https://showtime-git-fix-listed-for-showtime.vercel.app

@github-actions
Copy link

github-actions bot commented Jan 6, 2022

PR Preview - Storybook

This pull request preview deployment is now available.

✅ Preview: io.showtime.storybook://expo-development-client/?url=https://exp.host/@tryshowtime/showtime-storybook-react-native/index.exp?release-channel=pr-660&sdkVersion=44.0.0

Comment ID: 1006298924

@github-actions
Copy link

github-actions bot commented Jan 6, 2022

PR Preview - App

This pull request preview deployment is now available.

✅ Preview: io.showtime.development://expo-development-client/?url=https://exp.host/@tryshowtime/showtime/index.exp?release-channel=pr-660&sdkVersion=44.0.0

Comment ID: 1006300516

@github-actions
Copy link

github-actions bot commented Jan 6, 2022

PR Preview - Storybook on Chromatic

✅ Preview: The Storybook preview URL for this Pull Request

✅ Build: The Chromatic build URL for this Pull Request

Copy link
Contributor

@axeldelafosse axeldelafosse left a comment

Choose a reason for hiding this comment

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

LGTM

@TatisLois TatisLois merged commit 257a5ee into staging Jan 10, 2022
@TatisLois TatisLois deleted the fix/listed-for branch January 10, 2022 13:40
TatisLois added a commit that referenced this pull request Jan 17, 2022
* upgrade mmkv

* Home/Feed (#655)

* feed tabs

* pinch to zoom

* pinch to zoom

* pinch to zoom

* pinch to zoom

* fix: feed ptr

* fix: feed images

* fix: feed images

* wip: feed

* wip: feed

* wip: feed

* wip - feed

* wip - feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* wip: feed

* upgrade mmkv

* use correct app scheme and icon for walletconnect

* improve walletconnect implementation

* automatically submit when wallet is connected

* Adjust gas price by a percentage (#653)

* create video component

* Feature/show2 290 trending screen (#663)

* wip: trending

* wip: trending

* wip: trending

* wip: trending

* make api fast again

* optimise creator previre

* optimise creator preview

* optimise creator preview

* optimise creator preview

* Update to handle long collection names (#662)

* Enable Magic on minting & marketplace (#665)

* install three.js + upgrade to react 18 + patch packages

* add model component

* Misc fixes (#666)

* spinner fix

* spinner fix

* add swr

* add swr

* add swr

* initial scroll fix

* add accessToken hook (#667)

* fix: added `useAccessToken` to access token for web (#669)

* feat(SHOW2-225): refactor modal component (#652)

* refactor: updated the modal styling for web

* fix: fixed tailwind colors import

* refactor: updated the modal styling for mobile

* chore: reset modal storybook visibility

* refactor: removed index.web

* fix: fixed the modal body styling on mobile

* refactor: updated modal styling for web/ios

* fix: modal gestures components on Android

Co-authored-by: Axel Delafosse <[email protected]>

* list improvements (#671)

* list improvements

* list improvements

* list improvements

* list improvements

* list improvements

* list improvements

* list improvements

* render model as image for now

* mark svg as todo

* mark blurhash as todo on android

* update logo

* add nft title and description components

* clean code

* Update styles and semantics for listed for per feedback (#660)

* disable save changes button when username is not available (#674)

* use suspense + add types + add navigation elements context and hook to hide

* improve collection component

* add send icon

* add comments + message box

* use constants for scroll and padding height

* clean code

* chore(magic): add magic flag in useFlags and import them into modals (#675)

* Profile screen (#677)

* add flipper

* add flipper

* inti

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* enable double tap

* list improvements

* list improvements

* wip profile

* wip profile

* profile api

* profile wip

* profile wip

* profile wip

* profile wip

* profile wip

* profile wip

* profile wip

* wip profile

* wip profile

* wip profile

* wip profile

* wip profile

* wip profile

* profile change route param

* add react navigation to storybook RN

* add react navigation to storybook RN

* add react navigation to storybook RN

* add react navigation to storybook RN

* add camera

* add reanimated handler to pager (#678)

* add reanimated handler to pager

* add reanimated handler to pager

* new tabs

* Revert "new tabs"

This reverts commit b88ae0b.

* upgrade vision-camera

* add create modal

* clean code

* refactor: updated tabs item header width to be dynamic (#681)

* update dependencies

* upgrade deps and revert yarn.lock changes

* use text-13 for tab items

* render svg via xml + disabled press on media if in nft modal

* fix camera tab bar icon container size

* set trending tabs to full width screen

* use suspense for profile tabs

* clean header dropdown

* Show an error when username is too short

* set user color scheme

* update status bar

* fix imports

* Feature/show2 309 create (#688)

* fieldset

* wip create UI

* wip create UI

* wip create UI

* wip create UI

* remove animated view from header (#692)

* use media component everywhere

* use the modal component for the create screen

* fix: color mode re-renders

* fix: color mode re-renders

* fix: color mode re-renders

* feat(SHOW2-283): added login with SMS flow and refactor login styling (#672)

* fix: updated the button height based on design system

* refactor: updated login screen styling

* feat: added keyboard handling for iOS & Android

* refactor: extracted email field ui into a separate component

* feat: added phone number field

* refactor: added contact details and remove email and phone number components

* feat: added loading view

* chore: removed debug console logs

* fix: set loading state on phone number submitting

* refactor: extract root magic instance

* refactor: updated tabs header item indicator positioning

* refactor: added a generic input field

* refactor: extracted login container and header

* refactor: updated login design to follow design system

* fix: prevent tab header from setting invalid height

* refactor: added text content type to input field

* fix: logout customer when app fails to login

* fix(unlist): allow an item to be unlisted by the owner when it not the lowest item for sale (#683)

* Resolve an issue with next.js caching user profiles to a 404 (#680) (#698)

Co-authored-by: Axel Delafosse <[email protected]>

* rework explore page for marketplace (#700)

* rework explore page for marketplace

* fix mobile links

* fix feed button tracking events

Co-authored-by: Axel Delafosse <[email protected]>
Co-authored-by: Nishan <[email protected]>
Co-authored-by: Axel Delafosse <[email protected]>
Co-authored-by: Miguel Piedrafita <[email protected]>
Co-authored-by: Mo Gorhom <[email protected]>
Co-authored-by: Henry Fontanier <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants