Skip to content

Commit

Permalink
Release v0.2.41.
Browse files Browse the repository at this point in the history
Add missing read at date in book table.

Document the sheet service functions.

Make some lookup methods private.

Document more stuff.

Add E2E and refactor the sheet service methods.

Add E2E to validators.

Add E2E to ISBN.

Add missing hover state on search key tooltip.

Show keywords in search history items.

Add supports-backdrop-blur variant.

Remove loading indicator from table.

Change explorer transition.

Add a subtle ring to search dialog.

Disable book selection in mobile.

Improove a bit the UX of the search dialog.

Use skeleton utils in more components.

Fix search dialog jumping effect.

Add some WAI-ARIA attributes to search dialog.

Fix sheet test.

Bump the version to release.
  • Loading branch information
alessandrojean committed Feb 24, 2022
1 parent 80a140c commit 9f7ed96
Show file tree
Hide file tree
Showing 72 changed files with 4,268 additions and 746 deletions.
3 changes: 2 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@ module.exports = {
root: true,
env: {
browser: true,
node: true
node: true,
'jest/globals': true
},
extends: [
'plugin:vue/vue3-essential',
Expand Down
15 changes: 14 additions & 1 deletion babel.config.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,16 @@
module.exports = {
presets: ['@babel/preset-env']
presets: [['@babel/preset-env', { targets: { node: 'current' } }]],
env: {
test: {
plugins: [
() => ({
visitor: {
MetaProperty (path) {
path.replaceWithSourceString('process')
}
}
})
]
}
}
}
196 changes: 196 additions & 0 deletions jest.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,196 @@
/*
* For a detailed explanation regarding each configuration property, visit:
* https://jestjs.io/docs/configuration
*/

module.exports = {
// All imported modules in your tests should be mocked automatically
// automock: false,

// Stop running tests after `n` failures
// bail: 0,

// The directory where Jest should store its cached dependency information
// cacheDirectory: "/tmp/jest_rs",

// Automatically clear mock calls, instances and results before every test
// clearMocks: false,

// Indicates whether the coverage information should be collected while executing the test
// collectCoverage: false,

// An array of glob patterns indicating a set of files for which coverage information should be collected
// collectCoverageFrom: undefined,

// The directory where Jest should output its coverage files
// coverageDirectory: undefined,

// An array of regexp pattern strings used to skip coverage collection
// coveragePathIgnorePatterns: [
// "/node_modules/"
// ],

// Indicates which provider should be used to instrument code for coverage
// coverageProvider: "babel",

// A list of reporter names that Jest uses when writing coverage reports
// coverageReporters: [
// "json",
// "text",
// "lcov",
// "clover"
// ],

// An object that configures minimum threshold enforcement for coverage results
// coverageThreshold: undefined,

// A path to a custom dependency extractor
// dependencyExtractor: undefined,

// Make calling deprecated APIs throw helpful error messages
// errorOnDeprecated: false,

// Force coverage collection from ignored files using an array of glob patterns
// forceCoverageMatch: [],

// A path to a module which exports an async function that is triggered once before all test suites
// globalSetup: undefined,

// A path to a module which exports an async function that is triggered once after all test suites
// globalTeardown: undefined,

// A set of global variables that need to be available in all test environments
// globals: {},

// The maximum amount of workers used to run your tests. Can be specified as % or a number. E.g. maxWorkers: 10% will use 10% of your CPU amount + 1 as the maximum worker number. maxWorkers: 2 will use a maximum of 2 workers.
// maxWorkers: "50%",

// An array of directory names to be searched recursively up from the requiring module's location
// moduleDirectories: [
// "node_modules"
// ],

// An array of file extensions your modules use
// moduleFileExtensions: [
// "js",
// "jsx",
// "ts",
// "tsx",
// "json",
// "node"
// ],

// A map from regular expressions to module names or to arrays of module names that allow to stub out resources with a single module
moduleNameMapper: {
"^@/(.*)$": "<rootDir>/src/$1"
},

// An array of regexp pattern strings, matched against all module paths before considered 'visible' to the module loader
// modulePathIgnorePatterns: [],

// Activates notifications for test results
// notify: false,

// An enum that specifies notification mode. Requires { notify: true }
// notifyMode: "failure-change",

// A preset that is used as a base for Jest's configuration
// preset: undefined,

// Run tests from one or more projects
// projects: undefined,

// Use this configuration option to add custom reporters to Jest
// reporters: undefined,

// Automatically reset mock state before every test
// resetMocks: false,

// Reset the module registry before running each individual test
// resetModules: false,

// A path to a custom resolver
// resolver: undefined,

// Automatically restore mock state and implementation before every test
// restoreMocks: false,

// The root directory that Jest should scan for tests and modules within
// rootDir: undefined,

// A list of paths to directories that Jest should use to search for files in
// roots: [
// "<rootDir>"
// ],

// Allows you to use a custom runner instead of Jest's default test runner
// runner: "jest-runner",

// The paths to modules that run some code to configure or set up the testing environment before each test
// setupFiles: [],

// A list of paths to modules that run some code to configure or set up the testing framework before each test
// setupFilesAfterEnv: [],

// The number of seconds after which a test is considered as slow and reported as such in the results.
// slowTestThreshold: 5,

// A list of paths to snapshot serializer modules Jest should use for snapshot testing
// snapshotSerializers: [],

// The test environment that will be used for testing
testEnvironment: "jsdom",

// Options that will be passed to the testEnvironment
// testEnvironmentOptions: {},

// Adds a location field to test results
// testLocationInResults: false,

// The glob patterns Jest uses to detect test files
// testMatch: [
// "**/__tests__/**/*.[jt]s?(x)",
// "**/?(*.)+(spec|test).[tj]s?(x)"
// ],

// An array of regexp pattern strings that are matched against all test paths, matched tests are skipped
// testPathIgnorePatterns: [
// "/node_modules/"
// ],

// The regexp pattern or array of patterns that Jest uses to detect test files
// testRegex: [],

// This option allows the use of a custom results processor
// testResultsProcessor: undefined,

// This option allows use of a custom test runner
// testRunner: "jest-circus/runner",

// This option sets the URL for the jsdom environment. It is reflected in properties such as location.href
// testURL: "http://localhost",

// Setting this value to "fake" allows the use of fake timers for functions such as "setTimeout"
// timers: "real",

// A map from regular expressions to paths to transformers
// transform: undefined,

// An array of regexp pattern strings that are matched against all source file paths, matched files will skip transformation
// transformIgnorePatterns: [
// "/node_modules/",
// "\\.pnp\\.[^\\/]+$"
// ],

// An array of regexp pattern strings that are matched against all modules before the module loader will automatically return a mock for them
// unmockedModulePathPatterns: undefined,

// Indicates whether each individual test should be reported during the run
// verbose: undefined,

// An array of regexp patterns that are matched against all source file paths before re-running tests in watch mode
// watchPathIgnorePatterns: [],

// Whether to use watchman for file crawling
// watchman: true,
};
5 changes: 4 additions & 1 deletion jsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,8 @@
}
},
"include": ["./src/**/*"],
"exclude": ["node_modules"]
"exclude": ["node_modules"],
"typeAcquisition": {
"include": ["jest"]
}
}
14 changes: 12 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,13 +1,15 @@
{
"name": "toshokan",
"version": "0.2.40",
"version": "0.2.41",
"private": true,
"scripts": {
"dev": "vite",
"build": "vite build",
"serve": "vite preview",
"lint": "eslint src",
"lint:fix": "eslint src --fix"
"lint:fix": "eslint src --fix",
"test": "jest ./src/tests",
"test:watch": "jest --watch ./src/tests"
},
"dependencies": {
"@headlessui/vue": "^1.5.0",
Expand Down Expand Up @@ -41,19 +43,27 @@
"vuedraggable": "^4.1.0"
},
"devDependencies": {
"@babel/core": "^7.17.5",
"@babel/eslint-parser": "^7.17.0",
"@babel/preset-env": "^7.16.11",
"@types/dedent": "^0.7.0",
"@types/gapi.client.drive": "^3.0.13",
"@types/gapi.client.sheets": "^4.0.20201029",
"@types/google.visualization": "^0.0.68",
"@types/jest": "^27.4.0",
"@vitejs/plugin-vue": "^2.2.2",
"@vue/compiler-sfc": "^3.2.31",
"@vue/eslint-config-standard": "^6.1.0",
"autoprefixer": "^10.4.2",
"babel-jest": "^27.5.1",
"eslint": "^8.9.0",
"eslint-plugin-import": "^2.25.4",
"eslint-plugin-node": "^11.1.0",
"eslint-plugin-promise": "^6.0.0",
"eslint-plugin-standard": "^5.0.0",
"eslint-plugin-vue": "^8.4.1",
"git-describe": "^4.1.0",
"jest": "^27.5.1",
"postcss": "^8.4.6",
"postcss-loader": "^6.2.1",
"source-map": "^0.7.3",
Expand Down
10 changes: 6 additions & 4 deletions src/components/AppNavbar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@
enter-to-class="opacity-100"
>
<button
class="fake-search-input has-ring-focus"
class="fake-search-input has-ring-focus group"
v-if="!loading"
@click="showSearchDialog"
>
Expand Down Expand Up @@ -312,21 +312,23 @@ export default {

<style lang="postcss" scoped>
.app-navbar {
@apply fixed inset-x-0 bg-gray-800/95
@apply fixed inset-x-0
bg-gray-800 supports-backdrop-blur:bg-gray-800/95
backdrop-blur sm:backdrop-filter-none md:backdrop-blur
transition duration-300 ease-in-out sm:left-16 md:left-0;
}
.app-navbar.is-transparent {
@apply bg-transparent md:bg-gray-800/95;
@apply bg-transparent
supports-backdrop-blur:md:bg-gray-800/95 md:bg-gray-800;
}
.app-navbar.is-hidden:not(:focus-within) {
transform: translate3d(0, -100%, 0);
}
.app-navbar.is-hidden.is-transparent {
@apply bg-gray-800/95;
@apply supports-backdrop-blur:bg-gray-800/95 bg-gray-800;
}
.nav-link.is-active {
Expand Down
4 changes: 2 additions & 2 deletions src/components/GroupGrid.vue
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
aria-labelledby="groups-title"
class="w-full mb-3 mt-8"
>
<div v-if="sheetLoading || loading" class="motion-safe:animate-pulse h-6 bg-gray-400 dark:bg-gray-600 rounded w-40 mb-3"></div>
<div v-if="sheetLoading || loading" class="skeleton h-6 w-40 mb-3"></div>
<h2 v-else id="groups-title" class="font-medium font-display text-lg mb-2 dark:text-gray-200">
{{ t('dashboard.home.groups') }}
</h2>
Expand All @@ -16,7 +16,7 @@
<div
v-for="i in 4"
:key="i"
class="motion-safe:animate-pulse h-10 bg-gray-400 dark:bg-gray-600 rounded w-full"
class="skeleton h-10 w-full"
/>
</div>
<div
Expand Down
2 changes: 1 addition & 1 deletion src/components/LoadingIndicator.vue
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
:class="[
position,
zIndex,
'flex justify-center items-center inset-0 bg-opacity-75 bg-white dark:bg-gray-900 dark:bg-opacity-75',
'flex justify-center items-center inset-0 bg-white/75 dark:bg-gray-900/75',
blur ? 'backdrop-blur-sm' : ''
]"
v-if="loading"
Expand Down
2 changes: 1 addition & 1 deletion src/components/LocaleSelector.vue
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@
leave-from-class="opacity-100"
leave-to-class="opacity-0"
>
<ListboxOptions class="absolute z-10 mt-1 w-56 bg-white dark:bg-gray-700/80 dark:backdrop-blur shadow-lg max-h-56 rounded-md py-1 text-base border border-gray-200 dark:border-gray-600 overflow-auto focus:outline-none sm:text-sm">
<ListboxOptions class="absolute z-10 mt-1 w-56 bg-white dark:supports-backdrop-blur:bg-gray-700/80 dark:bg-gray-700 dark:backdrop-blur shadow-lg max-h-56 rounded-md py-1 text-base border border-gray-200 dark:border-gray-600 overflow-auto focus:outline-none sm:text-sm">
<ListboxOption
as="template"
v-for="loc in availableLocales"
Expand Down
Loading

0 comments on commit 9f7ed96

Please sign in to comment.