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

Prepare Release 1.15.0 #1005

Merged
merged 2 commits into from
Apr 18, 2017
Merged

Prepare Release 1.15.0 #1005

merged 2 commits into from
Apr 18, 2017

Conversation

giladgray
Copy link
Contributor

@giladgray giladgray commented Apr 18, 2017

🎆 Highlights: top quality bug fixes and some very impressive new Table features.

📖 Latest docs: blueprintjs.com/docs

@blueprintjs/core 1.15.0

@blueprintjs/datetime 1.13.0

@blueprintjs/table 1.12.0

  • 🌟 NEW drag-and-drop column and row reordering! [Table] drag and drop row re-ordering #213
    reordering-v2
    • Column and row reordering can be toggled independently via the following API:
      <Table isColumnReorderable={true} onColumnsReordered={myCallback} />
      <Table isRowReorderable={true} onRowsReordered={myCallback} />
    • Table defers to you to reorder your data appropriately. See the documentation for details on myCallback implementation; the new Utils.reorderArray function can be helpful here.
  • 🌟 NEW table.resizeRowsByTallestCell(columnIndex) instance method resizes all rows by the tallest visible cell in that column FR: Resize row height for currently visible items automatically on wrap text #929
    table-resize-tallest
    • all rows (even those offscreen) are resized to match the desired height (based on content) of the tallest currently visible cell in the given column index
    • to access this instance method you must first get a ref to your <Table>
  • 🌟 Fixed eliminate so many unnecessary re-renders [Table] Eliminate unnecessary re-renders everywhere #998
    • added @PureRender decorator to all pure table components
    • added custom shouldComponentUpdate logic where necessary to handle complex props
    • 🔥 RegionLayer internal component API accepts pre-computed regionStyles instead of getRegionStyle function (but you probably won't notice)

Documentation

@blueprint-bot
Copy link

Prepare Release v1.15.0

Preview: documentation | table | github pages
Coverage: core | datetime

@cmslewis
Copy link
Contributor

cmslewis commented Apr 18, 2017

Suggested additions to the Table reordering point:

  • 🌟 NEW drag-and-drop column and row reordering! [Table] drag and drop row re-ordering #213
    reordering-v2
    • Column and row reordering can be toggled independently via the following API:
      <Table isColumnReorderable={true} onColumnsReordered={myCallback} />
      <Table isRowReorderable={true} onRowsReordered={myCallback} />
    • Table defers to you to reorder your data appropriately. Use the new Utils.reorderArray function to help with this.

@cmslewis
Copy link
Contributor

nit (not strictly necessary, but reads a little clearer IMO):

We are tracking noticeable performance issues when scrolling tables that include truncated cells and are actively working on a fix.

@gscshoyru
Copy link
Contributor

gscshoyru commented Apr 18, 2017

Suggested edits to resizeRows point

  • 🌟 NEW table.resizeRowsByTallestCell(columnIndex) instance method resizes all rows by the tallest visible cell in that column.

apr-18-2017 23-33-38

  • should be used with word-wrapped cells, or cells that are otherwise vertical
  • will resize all rows in the table large enough to fit the largest currently visible cell in provided column index.
  • useful as a heuristic to resize all rows in a table large enough so all content that would otherwise be truncated will be visible as users scroll through the table.

@giladgray
Copy link
Contributor Author

removed performance warning cuz we haven't confirmed in prod mode react.

@cmslewis
Copy link
Contributor

cmslewis commented Apr 18, 2017

Huh, table-scrolling perf is actually much better in this preview than we observed yesterday, yet it's not in prod mode.

@giladgray giladgray merged commit 52de73a into master Apr 18, 2017
@giladgray giladgray deleted the release/1.15.0 branch April 18, 2017 22:15
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.

4 participants