Skip to content

Conversation

@MichaelDeBoey
Copy link
Contributor

Follow-up of #2806

Things I did extra:

  • Extract EditorTheme into its own component
  • Put all styles into the elements.ts file
  • Use FunctionComponent instead of React.FC, since it's the same, but FunctionComponent is a bit clearer I think

@MichaelDeBoey MichaelDeBoey added 🔨 Refactor 🧠 Overmind Indicates that this is related to the app's State Management labels Nov 22, 2019
@lbogdan lbogdan temporarily deployed to pr3074 November 22, 2019 15:20 Inactive
@lbogdan
Copy link
Contributor

lbogdan commented Nov 22, 2019

Build for latest commit 095c9d9 is at https://pr3074.build.csb.dev/s/new.

@christianalfoni
Copy link
Contributor

Hm, there seems to be some missing code and styling?

Screenshot 2019-11-28 at 09 41 43

Screenshot 2019-11-28 at 09 41 47

Are you running the project while refactoring? 😄 Let me know if you have any issues running it

Copy link
Contributor

@christianalfoni christianalfoni left a comment

Choose a reason for hiding this comment

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

Screenshots

@MichaelDeBoey
Copy link
Contributor Author

@christianalfoni This PR is waiting from before the #Hacktoberfest event.

I can't see the "Open VSCode Settings" button in production or my refactor, so I'm not sure how you can thb.

Will look into the width of the TextArea tho 🙂

@MichaelDeBoey
Copy link
Contributor Author

MichaelDeBoey commented Nov 28, 2019

I can't see the "Open VSCode Settings" button in production or my refactor, so I'm not sure how you can thb.

Found out this is because of the VSCodePlaceholder component, which needs store.preferences.settings.experimentVSCode.

Since there are no experiments atm, I can't test this out unfortunattely

There are no experiments running at the moment. Stay tuned for new
experiments!

Edit: on the other hand, If I didn't have store.preferences.settings.experimentVSCode, I should see children, which would need to show me a "Font Size" title... 😕

@lbogdan lbogdan temporarily deployed to pr3074 November 28, 2019 17:44 Inactive
@MichaelDeBoey
Copy link
Contributor Author

Found out the reason the "Open VSCode Settings" button isn't shown.

You need to be inside a sandbox to see it (so opening Preferences on dashboard won't show it):

<Route path={editorUrl()}>
{res =>
res.match && (
<div style={{ marginTop: '1rem' }}>
<Button small onClick={openCommand}>
Open VSCode Settings
</Button>
</div>
)
}
</Route>

export const editorUrl = () => `/s/`;

@MichaelDeBoey
Copy link
Contributor Author

@christianalfoni Resolved discussions + simplified components since settings.experimentVSCode is always true 🙂

@lbogdan lbogdan temporarily deployed to pr3074 November 28, 2019 18:31 Inactive
Copy link
Contributor

@christianalfoni christianalfoni left a comment

Choose a reason for hiding this comment

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

Looking great! 👍 😄

Tested:

  • Open appearance in settings
  • Open vscode
  • Tested custom color scheme

@christianalfoni christianalfoni merged commit 653c45f into codesandbox:master Dec 3, 2019
@MichaelDeBoey
Copy link
Contributor Author

Thanks! 🙂

@MichaelDeBoey MichaelDeBoey deleted the overmind/Appearance branch December 3, 2019 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧠 Overmind Indicates that this is related to the app's State Management 🔨 Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants