* feat(client): add duplicate account warning
* feat: check completed challenge count
* feat: stop redirecting /learn to /email-sign-up
* test: update to account for the lack of redirects
Also, in an extremely WET way, test both options.
* Update client/src/pages/email-sign-up.tsx
---------
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* feat(client): expose the language button to the navbar
* clean extra logic
Co-authored-by: Huyen Nguyen <25715018+huyenltnguyen@users.noreply.github.com>
* show the language menu
* set keyboard navigation logic
* change the language icon
* sort float values
* fix the test
* remove the menu roles from the languagelist
Co-authored-by: Bruce B <bbsmooth@gmail.com>
* add id to button and polish the attributes
Co-authored-by: Co-authored-by: Bruce B <bbsmooth@gmail.com>
* revert to production focus events
Co-authored-by: Co-authored-by: Bruce B <bbsmooth@gmail.com>
* fix the menu not closing and fix the label
Co-authored-by: Co-authored-by: Bruce B <bbsmooth@gmail.com>
* make event passing more explicit
Co-authored-by: Co-authored-by: Bruce B <bbsmooth@gmail.com>
* Let screenreaders annouce that the expand is closed
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
* let screenreader annouce list is closed
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
* polish the logic around the styling
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
* add comment about accessiblity function
* oh right active doesn't work like that
* hide the menu if the click is outside it
Co-authored-by: Bruce B <bbsmooth@gmail.com>
* fix the contrast in the check mark
Co-authored-by: Bruce B <bbsmooth@gmail.com>
* Stop the menu closing when clicking on the searchbar
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
* make the list hover and focus consistent
Co-authored-by: Co-authored-by: Bruce B <bbsmooth@gmail.com>
---------
Co-authored-by: Huyen Nguyen <25715018+huyenltnguyen@users.noreply.github.com>
Co-authored-by: Bruce B <bbsmooth@gmail.com>
* feat(client): stop margin from pushing outline out of the container
* remove the parent because it was breaking the layout on mobile devices
* sort the mobile and laptop layout
* remove extra property
* Update client/src/components/Header/components/universal-nav.css
* center the logo and fix the extra pixel issue
Co-authored-by: moT01 <20648924+moT01@users.noreply.github.com>
* increase padding top
---------
Co-authored-by: moT01 <20648924+moT01@users.noreply.github.com>
* fix: description if characters is over 288 characters on portfolio
* feat: add tests
* Update client/src/components/helpers/form/block-save-button.tsx
* feat: npm -> pnpm
This resolves the issues with the gatsby client (gatsby-plugin-pnpm
deals with the fact that gatsby is relying on its own dependencies
being de-duped)
and challenge-editor (which doesn't seem to want to automatically install
codemirror and needed its own eslint config)
* fix: correct mocha path for curriculum tests
* fix: use select workspace with -F not -w
* fix: reorganise packages and restrict hoisting
pnpm works best if the workspaces keep their own dependencies, since
dependencies are not flattened and then what node resolves from a
require is predictable.
@types seem to be a special case and more care is required to prevent
them getting smushed together in the root (hence the .npmrc)
* fix: add types for tools + root
* fix: decouple challenge-auditor from client
* fix: add ui-components types
* fix(client): use the latest types for react 16
* fix: prettify
* fix: prettierignore pnpm-lock
* fix: relax hoisting
Turns out pnpm works just fine with types. I don't know what was going
wrong before, but there are no-longer any type conflicts.
* fix: add @redux-saga/core to fix eslint issue
It seems to only be redux-saga that import/named can't cope with, so it
is probably okay to work around this one.
* chore: add chai to tools/scripts/build
* fix: add store to root for cypress
* fix: allow cypress to download binaries
If we want to keep preventing cypress from downloading binaries, we can
figure out a workaround, but I'm allowing it to ease the transition to
pnpm.
My guess about why this is happening is that npm triggers Cypress's
postinstall script, but pnpm does not (because pnpm install only
installs if necessary, perferring to link)
* chore: re-enable pre/post scripts
* fix: update build scripts for client
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* chore: update engines to use pnpm
* fix: enable choice of (super)block for tests
Only 'nix machines for now.
* chore: pin pnpm to version 7
* chore: remove last npms
Except web + curriculum-server. I'll update them when I start work on
them again.
* fix: lockfile check to catch any package-locks
* fix(action): install pnpm for upcoming tests
* chore: add nodemon to new api
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* feat(client): change the buttons from disabled to aria-disabled
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
* stop the API call when the validation isn't true
* use aria disabled instead of disabled on the test?
* use have.attr to check for aria disabled
* hide the button when it's disabled
Co-authored-by: Bruce B <bbsmooth@gmail.com>
* update Privacy Settings save button
---------
Co-authored-by: Bruce Blaser <bbsmooth@gmail.com>
