Skip to content

Commit 8085cd8

Browse files
jassmithmarkmiro
andauthored
5.1.0 (#486)
* Fix rare bug that can cause a flickering line * Add overrideable header border bottom color * Add onKeyDown/Up * Allow disabling drawing of the focus ring * Allow passing of custom image window loader * Fix markdown button * Custom renderer API (#481) * Update names * Add third party renderer API * "Fix" build * More correct type * Implement onPaste * Unify draw args * Mostly working? * Fix build * This one too * Cleanup most casts * Fix tests * Deprecate useCustomCells * Improve type compat * Fix * Add test for broken space behavior * Fix test * Make it possible to disable grow when resizing * Move to NPM workspaces (#482) * Building workspaces * Storybook crap * Fix storybook * Cleanup versioning * Hopefully fix CI * Try again * Correct node version * Simplify * More work on correct builds * More build fixes * Fix broken tests * 5.0.1-alpha1 * Try fix deploy * Update prod scripts * Add precise scrolling and tests * Start/center/end tests * Simplify type definition * Add links cell * Fix buggy sdouble scrolling behavior when using a controlled selection * Re-enable code splitting * Also enable splitting for cells package * Fix deps culling * Fix naming * Remove resolution * Fix error where links would still open editor eroniously * 5.0.1-alpha2 * Add link cell editor * Remove do nothing code * Remove more dead code * Offer better control over propagation * Add tests for weird behavioral issue with scrolling * Minor bump * Increase permisivity of ImageWindowLoader interface * Add control for image cell rounding radius * Improve radius further * Fix focus ring * add ability to provide initial grid size * Start adding support for scrollOffsetX/Y * Fix test * Support bounce scrolling * Duh * 5.1.0-alpha2 * Remove random math.random that was left in from debugging * Add button cell * Clean * Improve button behavior * Do better math * Fix middle center bias fetch * Enforce hover requriements * add ability to independtly control max auto column size and max user settable column size * docs v2 (#488) * Start working on docs v2 * More docs * More docs * Update docs * Fix drawing bug with draw focus ring disabled * Fix initial scroll offset working when no initial size provided * Fix issue where scrolling to invalid locations would sometimes fail weirdly * Implement visible region x watching as well * 5.1.0-alpha3 * Improve types to reduce errors * Fix perf issue with data grid * make it easier to see perf issues in storybook * Cut an annoying amount of CPU time off the render loop for high FPS displays * Fix tests * 5.1.0-alpha4 * Minor perf improvements * Fix build * Add tests * Disable blitting in safari as they broke the fast path * 5.1.0-alpha5 * Fix infinite scroll * Make diagonal scroll prevention more reliable and less jumpy * More type improvements * Double buffer support for safari (#491) * Basic buffer juggling * Make double buffering an option * Ensure row markers are stable * Make double buffering configurable * Allow selection from all possible blit modes * Fix tests * Build fixes * Resolve issue where dropdown cell would select first item every time regardless of what is selected * Doo dee doo * Add some tests for compact selection * One more test * Handle scaling factors * Fix tests * Fix classname being passed to canvas inappropriately * 5.1.0-beta1 * Export declaration maps * Fix keyboard shortcut not focusing search input (#443) If search field loses focus, pressing the keyboard shortcut again should refocus it. Not addressing customization in: #443 * Prep for stable Co-authored-by: Mark Miro <[email protected]>
1 parent 6409366 commit 8085cd8

File tree

113 files changed

+45561
-16522
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

113 files changed

+45561
-16522
lines changed

.github/workflows/beta.js.yml

Lines changed: 8 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,10 @@ jobs:
1818
- uses: actions/checkout@v1
1919
- uses: actions/setup-node@v1
2020
with:
21-
node-version: 14
22-
- name: Bootstrap
23-
run: npm run bootstrap
21+
node-version: 16
22+
- name: Build
23+
run: npm install && npm run build
2424
working-directory: .
25-
- run: npm run lint
26-
- run: npm run build-js
27-
- run: npm run build-types
2825
- uses: JS-DevTools/npm-publish@v1
2926
with:
3027
token: ${{ secrets.NPM_TOKEN }}
@@ -40,13 +37,10 @@ jobs:
4037
- uses: actions/checkout@v1
4138
- uses: actions/setup-node@v1
4239
with:
43-
node-version: 14
44-
- name: Bootstrap
45-
run: npm run bootstrap
40+
node-version: 16
41+
- name: Build
42+
run: npm install && npm run build
4643
working-directory: .
47-
- run: npm run lint
48-
- run: npm run build-js
49-
- run: npm run build-types
5044
- uses: JS-DevTools/npm-publish@v1
5145
with:
5246
token: ${{ secrets.NPM_TOKEN }}
@@ -62,14 +56,10 @@ jobs:
6256
- uses: actions/checkout@v1
6357
- uses: actions/setup-node@v1
6458
with:
65-
node-version: 14
59+
node-version: 16
6660
- name: Bootstrap
67-
run: npm run bootstrap
61+
run: npm install && npm run build
6862
working-directory: .
69-
- run: npm run lint
70-
- run: npm run build-js
71-
- run: npm run build-cjs
72-
- run: npm run build-types
7363
- uses: JS-DevTools/npm-publish@v1
7464
with:
7565
token: ${{ secrets.NPM_TOKEN }}

.github/workflows/node.js.yml

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,13 @@ on: push
88
jobs:
99
build:
1010
runs-on: ubuntu-latest
11-
strategy:
12-
matrix:
13-
node-version: [14.x]
14-
1511
steps:
1612
- uses: actions/checkout@v2
17-
- name: Use Node.js ${{ matrix.node-version }}
13+
- name: Use Node.js 16
1814
uses: actions/setup-node@v1
1915
with:
20-
node-version: ${{ matrix.node-version }}
21-
- run: npm run bootstrap
16+
node-version: 16
17+
- run: npm install
2218
- run: npm run build
2319
- run: npm run test -- --coverage
2420
- run: npm run test-source
@@ -35,6 +31,6 @@ jobs:
3531
- uses: actions/checkout@v1
3632
- uses: actions/setup-node@v1
3733
with:
38-
node-version: 14
39-
- run: npm run bootstrap
34+
node-version: 16
35+
- run: npm install
4036
- run: npm run test-18

.github/workflows/release.js.yml

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,10 @@ jobs:
1919
- uses: actions/checkout@v1
2020
- uses: actions/setup-node@v1
2121
with:
22-
node-version: 14
22+
node-version: 16
2323
- name: Bootstrap
24-
run: npm run bootstrap
24+
run: npm install && npm run build
2525
working-directory: .
26-
- run: npm run lint
27-
- run: npm run build-js
28-
- run: npm run build-types
2926
- uses: JS-DevTools/npm-publish@v1
3027
with:
3128
token: ${{ secrets.NPM_TOKEN }}
@@ -40,13 +37,10 @@ jobs:
4037
- uses: actions/checkout@v1
4138
- uses: actions/setup-node@v1
4239
with:
43-
node-version: 14
40+
node-version: 16
4441
- name: Bootstrap
45-
run: npm run bootstrap
42+
run: npm install && npm run build
4643
working-directory: .
47-
- run: npm run lint
48-
- run: npm run build-js
49-
- run: npm run build-types
5044
- uses: JS-DevTools/npm-publish@v1
5145
with:
5246
token: ${{ secrets.NPM_TOKEN }}
@@ -61,14 +55,10 @@ jobs:
6155
- uses: actions/checkout@v1
6256
- uses: actions/setup-node@v1
6357
with:
64-
node-version: 14
58+
node-version: 16
6559
- name: Bootstrap
66-
run: npm run bootstrap
60+
run: npm install && npm run build
6761
working-directory: .
68-
- run: npm run lint
69-
- run: npm run build-js
70-
- run: npm run build-cjs
71-
- run: npm run build-types
7262
- uses: JS-DevTools/npm-publish@v1
7363
with:
7464
token: ${{ secrets.NPM_TOKEN }}

.github/workflows/storybook.js.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@ jobs:
1313
persist-credentials: false
1414
- uses: actions/setup-node@v1
1515
with:
16-
node-version: 14
16+
node-version: 16
1717
- name: Install and Build 🔧
1818
run: |
19-
npm run bootstrap
19+
npm install
2020
npm run build-storybook
2121
- name: Deploy 🚀
2222
uses: JamesIves/[email protected]

.gitignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,6 @@ node_modules*
66
.cache
77
pubsub-debug.log
88
storybook-build/
9-
coverage/
9+
coverage/
10+
*.tsbuildinfo
11+
**/docs/*

.npmrc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
legacy-peer-deps=true

.nvmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
v14.18.0
1+
v16.16.0

.storybook/main.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
const MiniCssExtractPlugin = require("mini-css-extract-plugin");
22

33
module.exports = {
4-
stories: ["../**/*.stories.tsx"],
5-
addons: [
6-
"@storybook/addon-links",
7-
"@storybook/addon-essentials",
8-
"@storybook/addon-knobs",
9-
"@storybook/addon-controls",
10-
],
4+
stories: ["../**/src/**/*.stories.tsx"],
5+
addons: ["@storybook/addon-controls"],
116
core: {
127
builder: "webpack5",
138
},
9+
typescript: {
10+
reactDocgen: false,
11+
},
1412
webpackFinal: async (config, { configType }) => {
1513
// `configType` has a value of 'DEVELOPMENT' or 'PRODUCTION'
1614
// You can change the configuration based on that.
@@ -33,6 +31,8 @@ module.exports = {
3331
],
3432
});
3533

34+
config.optimization.minimize = false;
35+
3636
return config;
3737
},
3838
};

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ is to use Codespaces if you have access to it. If you do not, simply cloning the
1515
Once codespaces is up and running make sure `jq` is installed and then:
1616

1717
```bash
18-
npm run bootstrap && npm run storybook
18+
npm run install && npm run storybook
1919
```
2020

2121
## Forking the data grid?

README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
<p align="center">Built as the basis for the <a href="https://www.glideapps.com/data-editor" target="_blank">Glide Data Editor</a>. <a href="https://www.glideapps.com/jobs#open-positions" target="_blank">We're hiring</a>.</p>
88

9-
109
<picture>
1110
<source media="(prefers-color-scheme: dark)" srcset="https://raw.githubusercontent.com/glideapps/glide-data-grid/master/media/data-grid-dark.png">
1211
<source media="(prefers-color-scheme: light)" srcset="https://raw.githubusercontent.com/glideapps/glide-data-grid/master/media/data-grid.png">
@@ -86,14 +85,14 @@ function getData([col, row]: Item): GridCell {
8685
kind: GridCellKind.Text,
8786
data: person.firstName,
8887
allowOverlay: false,
89-
displayData: person.firstName
88+
displayData: person.firstName,
9089
};
9190
} else if (col === 1) {
9291
return {
9392
kind: GridCellKind.Text,
9493
data: person.lastName,
9594
allowOverlay: false,
96-
displayData: person.lastName
95+
displayData: person.lastName,
9796
};
9897
} else {
9998
throw new Error();
@@ -103,7 +102,7 @@ function getData([col, row]: Item): GridCell {
103102

104103
## Full API documentation
105104

106-
The full [API documentation is in the `API.md` file](packages/core/API.md).
105+
The full [API documentation is on the main site](https://grid.glideapps.com/docs/index.html).
107106

108107
# 📒 FAQ
109108

@@ -152,6 +151,7 @@ There are some hacks you can do like setting timers and entering into a "low fid
152151
The easiest way to use the grid with Next is to create a component which wraps up your grid and then import it as a dynamic.
153152

154153
home.tsx
154+
155155
```tsx
156156
import type { NextPage } from "next";
157157
import dynamic from "next/dynamic";
@@ -177,6 +177,7 @@ export const Home: NextPage = () => {
177177
```
178178

179179
grid.tsx
180+
180181
```tsx
181182
import React from "react";
182183
import DataEditor from "@glideapps/glide-data-grid";

bootstrap.sh

Lines changed: 0 additions & 32 deletions
This file was deleted.

build-all.sh

Lines changed: 0 additions & 11 deletions
This file was deleted.

0 commit comments

Comments
 (0)