Skip to content

Conversation

@Adamj1232
Copy link
Contributor

@Adamj1232 Adamj1232 commented Aug 2, 2022

Description

This PR adds the ability to add a configuration to hide the Connect Modal side bar.
The changes adds a connect property on the AppState and init options with an internal prop of showSidebar : boolean that defaults to true

Screen Shot 2022-08-02 at 5 16 34 PM

Checklist

  • The version field in package.json is incremented following semantic versioning
  • 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 type-check & yarn build to confirm there are not any associated errors
  • This PR passes the Circle CI checks

@Adamj1232 Adamj1232 requested review from lnbc1QWFyb24, mahmud-bn and taylorjdawson and removed request for lnbc1QWFyb24 and taylorjdawson August 2, 2022 23:16
@Adamj1232 Adamj1232 changed the title Add configuration to hide/show sidebar [core-v2.7.0-alpha.1, react-v2.2.6-alpha.1, vue-v2.1.6-alpha.1] : Feature - Add configuration to hide/show sidebar Aug 2, 2022
Copy link
Contributor

@lnbc1QWFyb24 lnbc1QWFyb24 left a comment

Choose a reason for hiding this comment

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

Just a couple of suggestions, but nothing crucial to the PR.

@Adamj1232 Adamj1232 merged commit e27f53c into v2-web3-onboard-develop Aug 3, 2022
@Adamj1232 Adamj1232 deleted the feature/hide_sidebar_config branch August 3, 2022 14:21
@Adamj1232 Adamj1232 mentioned this pull request Aug 3, 2022
5 tasks
@lnbc1QWFyb24 lnbc1QWFyb24 mentioned this pull request Aug 12, 2022
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.

3 participants