Skip to content

Conversation

@Adamj1232
Copy link
Contributor

@Adamj1232 Adamj1232 commented Feb 17, 2023

Description

This PR adds the configurable "I don't have a wallet" link to the sidebar of the connect modal along with fixing the progress bar to match the designs (specifically for desktop as mobile matched already)

The "I don't have a wallet" link is configurable within the connect object inside the onboard config

type ConnectModalOptions = {
  showSidebar?: boolean
  /**
   * Disabled close of the connect modal with background click and
   * hides the close button forcing an action from the connect modal
   */
  disableClose?: boolean // defaults to false
  /**If set to true, the last connected wallet will store in local storage.
   * Then on init, onboard will try to reconnect to that wallet with
   * no modals displayed
   */
  autoConnectLastWallet?: boolean // defaults to false
  /**
   * Customize the link for the `I don't have a wallet` flow shown on the
   * select wallet modal.
   * Defaults to `https://ethereum.org/en/wallets/find-wallet/#main-content`
   */
  iDontHaveAWalletLink?: string
}

Screen Shot 2023-02-17 at 10 09 27 AM

Screen Shot 2023-02-17 at 10 09 02 AM

Screen Shot 2023-02-17 at 10 08 27 AM

Checklist

  • The version field in package.json of the package you have made changes in is incremented following semantic versioning and using alpha release tagging
  • The box that allows repo maintainers to update this PR is checked
  • I tested locally to make sure this feature/fix works
  • I have run yarn file-check, yarn type-check & yarn build to confirm there are not any associated errors
  • This PR passes the Circle CI checks

@vercel
Copy link

vercel bot commented Feb 17, 2023

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

Name Status Preview Comments Updated
web3-onboard-docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Feb 21, 2023 at 6:33PM (UTC)

@leightkt
Copy link
Contributor

leightkt commented Feb 21, 2023

@Adamj1232
Code overall is looking good- was able to add a custom link and also test the default link

When I test in the demo- the three dots below the "I don't have a wallet" link are not spacing as nicely as you in your demo pics:
Screenshot 2023-02-21 at 11 57 10 AM
Screenshot 2023-02-21 at 11 57 32 AM

@Adamj1232
Copy link
Contributor Author

@katspaugh thanks for catching that! Needed to push the commit removing a sneaky single quote breaking everything! Should be good now

Copy link
Contributor

@leightkt leightkt left a comment

Choose a reason for hiding this comment

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

Looks good- was able to test locally and add a custom link and test the default

@Adamj1232 Adamj1232 removed the request for review from gesquinca February 21, 2023 18:28
@Adamj1232 Adamj1232 merged commit 6031cbe into develop Feb 22, 2023
@Adamj1232 Adamj1232 deleted the feature/no_wallet_flow branch February 22, 2023 21:54
Adamj1232 added a commit that referenced this pull request Feb 28, 2023
* update root path (#1543)

* Add XDEFI module and injected to docs site (#1544)

* Add xdefi to docs

* Add xdefi module to docs

* Add xdefi package to docs onboard

* Add to injected wallets list

* Feature: "I don't have a wallet" flow + Progress-bar fixup  (#1535)

* Link working and styled

* Progress bar work

* Cleanup progressbar code

* Remove single quote breaking view

* Revert TP testing code

* add transaction preview graphic

* add transaction preview install code snippet

* move md code snippets

- update imports

* add transaction preview section to landing page

* update theming code snippet

* update hero section install code

* update account center graphic

* update hero w3o graphic

* update account center svg

* Update README.md (#1547)

Yeah maybe this needs more thorough changes throughout the docs but I was caught up on this for a minute, and this will help

* move transaction preview section to top

* update link url

* Added XDEFI to readme

* Add theming docs, refine core init options, api actions and code chunks (#1548)

* Add theming docs, still needs refinement

* More docs

* Add theming video

* More docs

* Update layout and add copy to some code chunks

* Add dividers for better visibility between options and copy to code blocks where useful

* Reorder Theming docs layout

* Add note about ethers v6 usage

* edit theming page

- add css variables table
- edit theme descriptions
- re-org existing content

* Update docs/src/lib/components/examples/connect-wallet/ReactConnectWallet.md

* Update theming tool to default to custom

---------

Co-authored-by: Gustavo Esquinca <[email protected]>

* Update docs/src/lib/components/code-snippet/theming-code.md

Co-authored-by: Adam Carpenter <[email protected]>

* Bump versions for release

* Add Infinity Wallet Connector (#1540)

* Add Infinity Wallet Connector

* Update packages/infinity-wallet/package.json

* Update packages/infinity-wallet/src/index.ts

* Remove yarn lock from infin package

* Update packages/demo/package.json

* Add inifinity to docs

* Reorder wallets

* Update docs title

* Bump innjected package version

---------

Co-authored-by: Adam Carpenter <[email protected]>

* [walletconnect-2.3.0-alpha.3] - Feat: Walletconnect URI handler + bump wc package version (#1552)

* accept handleUri prop

* Update docs

* Rename wc prop to handleUri

* Bump wc pakcage version for walletconnect/ethereum-provider

* merge in dev

* Revert unintended change

* Update prop in docs

* Cleanup semicolon

* Yarn it!

* Bump undici from 5.10.0 to 5.19.1 in /examples/with-sveltekit (#1532)

Bumps [undici](https://github.com/nodejs/undici) from 5.10.0 to 5.19.1.
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v5.10.0...v5.19.1)

---
updated-dependencies:
- dependency-name: undici
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam Carpenter <[email protected]>

* Update fortmatic docs

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Taylor Dawson <[email protected]>
Co-authored-by: Gustavo Esquinca <[email protected]>
Co-authored-by: Brent <[email protected]>
Co-authored-by: mhchaudhry3 <[email protected]>
Co-authored-by: mhchaudhry3 <[email protected]>
Co-authored-by: Aaron <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@Adamj1232 Adamj1232 mentioned this pull request Feb 28, 2023
5 tasks
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.

4 participants