Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

Fela Migration #100

Closed
8 of 33 tasks
jwineman opened this issue Mar 16, 2017 · 11 comments
Closed
8 of 33 tasks

Fela Migration #100

jwineman opened this issue Mar 16, 2017 · 11 comments

Comments

@jwineman
Copy link
Contributor

jwineman commented Mar 16, 2017

Lets track the migration of these components here. Call out what you're working on and I'll keep this thread up to date.

8 / 33 ( 24% ) complete.

@manatarms
Copy link
Contributor

cf-component-page please!

@wyuenho
Copy link
Contributor

wyuenho commented Mar 17, 2017

Please stop merging into master for now. We agree on how to support a "stable" version of cf-ui and work on big changes like this.

@manatarms
Copy link
Contributor

cf-component-label in PR #103

@tajo
Copy link
Contributor

tajo commented Mar 17, 2017

Afaik, it should be fine to merge things. Just don't forget to major bump the new components when you are publishing them for the first time. We can still do bug fix releases of older versions.

@tajo
Copy link
Contributor

tajo commented Mar 24, 2017

Please check out this on our wiki: WWW / JavaScripts / How to update cf-ui components to Fela

@tajo
Copy link
Contributor

tajo commented Mar 24, 2017

Note

List of all cf-component cross-dependencies (fortunately it's not that common)

  • cf-component-card -> cf-component-link
  • cf-component-copyable-textarea -> cf-component-textarea
  • cf-component-dropdown -> cf-component-dropdown
  • cf-component-loading -> cf-component-icon
  • cf-component-modal -> cf-component-heading and cf-component-icon
  • cf-component-pagination -> cf-component-icon
  • cf-component-progress -> cf-component-link
  • cf-component-tabs -> cf-component-select and cf-component-viewport

This means that for components

  • cf-component-link
  • cf-component-textarea
  • cf-component-dropdown
  • cf-component-icon
  • cf-component-heading
  • cf-component-select
  • cf-component-viewport

we need to be extra careful that they are not broken once released because they will "silently" appear in our projects everywhere where the card, dropdown... etc are used. So keep an extra eye on them and make sure they match our current styles!

The rest of components will (or can) be introduced slowly by replacing imports like cf-component-xxx with @cloudflare/cf-component-xxx. That can be even done on instance by instance basis.

@koddsson
Copy link
Contributor

koddsson commented Apr 3, 2017

I updated the description directly. Hope that's cool.

koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 19, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 19, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 19, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
The `ModalClose` component uses the `Icon` component so we need to
update the snapshots as well as wrapping the `ModalClose` tests in fela
context so it can render correctly.

Related-to: cloudflare#100
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 22, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 23, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 23, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 23, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 23, 2017
koddsson pushed a commit to koddsson/cf-ui that referenced this issue May 23, 2017
Migrate the `cf-component-icon` to ship with it's styles implemented
using the "css-in-js" framework `fela`.

Related-to: cloudflare#100
@grimunit
Copy link

Please check out this on our wiki: WWW / JavaScripts / How to update cf-ui components to Fela

Hey @tajo I'm probably just dumb and missing something obvious but I can't seem to find what wiki you're referring to. I'd like to look at that documentation.

@koddsson
Copy link
Contributor

@grimunit @tajo is referring to our internal wiki at Cloudflare :)

@grimunit
Copy link

@koddsson ok haha, that makes sense. Just interested in how you guys are reconciling existing components with Fela. Looking through your commits on the ones you've converted so far. Implementing something like the cf-style-container will probably make passing custom fela class rules to existing material-ui components a much smoother process. You guys are doing some cool work with Fela 😄

@koddsson
Copy link
Contributor

In order for to track better I've generated a indvidual issue for each of the remaining components to migrate and the a "Fela Migration" milestone so we don't have to manually update this issue :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests