Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

what about gdAlignRows, and other gd* attributes from flex-layout? #9

Open
seyfer opened this issue Apr 5, 2024 · 6 comments
Open

Comments

@seyfer
Copy link

seyfer commented Apr 5, 2024

is it planned to support?
there is a hint: angular/flex-layout#1426 (comment)

@synopss
Copy link
Owner

synopss commented Apr 6, 2024

Yes, I didn't have much time to work on this lately as you can tell, but this CLI will support all attributes that are possible to migrate (because some aren't sadly).

@seyfer
Copy link
Author

seyfer commented Apr 7, 2024

yep, this is a great starting tool, then what I do is

  1. I keep the flex-layout package installed because some fx* attributes are either stay in the code or not converted properly
  2. I go page by page and review the conversion, fixing and converting the remaining fx* attributes
  3. also converting gd* manually
  4. when all done - uninstall flex-layout

just having the gd* implemented would help a bit with this process. thanks!

@synopss
Copy link
Owner

synopss commented Apr 7, 2024

yep, this is a great starting tool, then what I do is

1. I keep the flex-layout package installed because some fx* attributes are either stay in the code or not converted properly

2. I go page by page and review the conversion, fixing and converting the remaining fx* attributes

3. also converting gd* manually

4. when all done - uninstall flex-layout

just having the gd* implemented would help a bit with this process. thanks!

Can you write down there which fx attributes are not converted properly please ?

@seyfer
Copy link
Author

seyfer commented Apr 7, 2024

@synopss I can give some examples, while I still migrate it at one of the projects. When I run CLI, these are pieces of code that I have left, and I go fix them manually.

<div
      fxLayoutAlign="start"
      fxLayoutAlign.md="center"
      fxLayoutAlign.sm="center"
      class="xs:flex xs:flex-col xs:justify-center xs:content-stretch xs:items-stretch xs:w-full flex flex-row"
    >

here probably only fxLayoutAlign.xs="center" was handled, but the rest left as it was before

<div
    mat-subheader
    gdAlignRows="stretch"
    class="job-header flex justify-between content-end items-end sm:flex sm:flex-col gap-[5px] h-full w-full min-h-full min-w-full"
    fxLayout="row"
    fxLayout.xs="column"
    fxLayoutAlign.xs="space-between"
    fxLayoutAlign.sm="space-between"
  >
<div
  gdAlignRows="stretch"
  fxLayout="row"
  fxLayout.xs="column"
  class="sm:flex sm:flex-col flex gap-2.5 h-full w-full min-h-full min-w-full"
>

hope this helps.

@synopss
Copy link
Owner

synopss commented Apr 7, 2024

I see, so looks like it only handles the very first breakpoint they keep the rest.

@seyfer
Copy link
Author

seyfer commented Apr 8, 2024

One more example. Converted but not removed

<div fxShow.gt-md class="gt-md:basis-[35%] hidden gt-md:block">

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

No branches or pull requests

2 participants