Skip to content

Fil/grid facet - #293

Closed
Fil wants to merge 5 commits into
mainfrom
fil/grid-facet-277
Closed

Fil wants to merge 5 commits into
mainfrom
fil/grid-facet-277

Conversation

@Fil

@Fil Fil commented Apr 2, 2021

Copy link
Copy Markdown
Contributor

for #277

todo:

  • automatic removal of the fx/fy fake axes
  • cleaner method to add the facets' ticks in a proper place

pending question

  • is x === y the correct test? if not, we can replace this easily by facet: {data, xy: accessor} or something.

Capture d’écran 2021-04-02 à 16 15 12

@Fil
Fil requested a review from mbostock April 2, 2021 14:15
@Fil Fil changed the title Fil/grid facet 277 Fil/grid facet Apr 2, 2021
Fil added 4 commits April 13, 2021 18:50
We still have to struggle a bit to remove the automatic (and now meaningless) facet labels, and position our own facet labels in each frame.
@Fil
Fil force-pushed the fil/grid-facet-277 branch from fd16d0a to 6580a38 Compare April 13, 2021 17:04
@Fil
Fil marked this pull request as ready for review April 13, 2021 17:06
Fil added a commit that referenced this pull request Apr 19, 2021
@Fil Fil mentioned this pull request Apr 19, 2021
3 tasks
@Fil

Fil commented Apr 19, 2021

Copy link
Copy Markdown
Contributor Author

superseded by #332

@Fil Fil closed this Apr 19, 2021
@Fil
Fil deleted the fil/grid-facet-277 branch April 19, 2021 15:22
Fil added a commit that referenced this pull request May 25, 2021
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.

1 participant