Skip to content

Update-output-only-branch #907

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

Merged
merged 45 commits into from
Jan 26, 2024
Merged

Update-output-only-branch #907

merged 45 commits into from
Jan 26, 2024

Conversation

sra405
Copy link
Contributor

@sra405 sra405 commented Jan 26, 2024

No description provided.

loiswells97 and others added 30 commits December 8, 2023 14:54
- Utilisting existing filename from `kramdown` for the code block
- Making the instructions progress bar more in line with the designs
Part of
[#2428](RaspberryPiFoundation/projects-ui#2428)

- css strings can be passed to the WebComponent through a `hostStyle`
prop, allowing the ShadowDOM to receive styles from the host app.
closes #775

---------

Co-authored-by: sra405 <[email protected]>
Co-authored-by: Scott <[email protected]>
Co-authored-by: Scott Adams <[email protected]>
closes #778

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #817

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #786

---------

Co-authored-by: magdalenajadach <[email protected]>
Co-authored-by: Magdalena Jadach <[email protected]>
- adds a custom event to notify Projects when the quiz is loaded
closes #813

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #783

---------

Co-authored-by: maxelkins <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #804

Changes:
- added `loadRemix` functionality
- user state comes from separate `localStorageUser` due to delays in
updating state
- if logged in `loadRemix` functionality will be called once
- if `loadRemix` fails it falls back to loading the base starter code
- loading the cache is conditional on either `remixLoad` failing on now
user object being set

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Scott <[email protected]>
closes #815

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: MagdalenaJadach <[email protected]>
closes #826

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #838

Notes:

Only an issue on Firefox so check multiple browsers with sidebar open
and closed

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Scott <[email protected]>
Co-authored-by: Scott Adams <[email protected]>
closes #836

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
…854)

closes #840

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #839

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #743

The idea is to intercept editor actions (i.e. actions that could call
the API) and update the user object in state prior to that call being
made. That way if the user object changes for whatever reason (like cred
renewal) the latest user object will be used for the API call
closes #846

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #834

- ProgressBar is added to quizzes
- Quizzes increment the currentStepPosition when they're complete so
that the next step can be rendered
- Quiz logic refactored to remove a useEffect() and `quizReady` state

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Pete Simonovic <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
closes #842

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
<img width="412" alt="Screenshot 2023-12-21 at 15 18 42"
src="https://github.com/%3Ca%20href="https://github.com/RaspberryPiFoundation/editor-ui/assets/88904316/dab26538-ccae-4caa-9efd-9944a2cc1500">https://github.com/RaspberryPiFoundation/editor-ui/assets/88904316/dab26538-ccae-4caa-9efd-9944a2cc1500">


closes #845

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: MagdalenaJadach <[email protected]>
closes #841

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: MagdalenaJadach <[email protected]>
closes #844

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: MagdalenaJadach <[email protected]>
Currently due to the styling changes required to maximise the height of
the web component in Firefox the standalone editor took up too much
space (didn't account for the top nav). This fix applies the required
`min-block-size` only to the web component
closes #814

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Scott <[email protected]>
Co-authored-by: Scott Adams <[email protected]>
Before:
![Screenshot 2024-01-04 at 15 41
01](https://github.com/RaspberryPiFoundation/editor-ui/assets/88904316/03107cc5-4729-4ee5-a747-b4a115fd1d4c)

After:
<img width="321" alt="Screenshot 2024-01-04 at 15 41 23"
src="https://github.com/%3Ca%20href="https://github.com/RaspberryPiFoundation/editor-ui/assets/88904316/3502d136-36bc-42d5-9f27-63c721df7f7b">https://github.com/RaspberryPiFoundation/editor-ui/assets/88904316/3502d136-36bc-42d5-9f27-63c721df7f7b">
Project site integration changes
sra405 and others added 14 commits January 11, 2024 10:03
This PR fixes and extends the assets url/blob replacements throughout
css and html:

- Allows multiple url replacements in css
- Generalises the attribute replacement and applies to `data-src`
attributes too
closes #829

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Fix so anything other than explicitly setting the boolean attributes as
false is taken as true
closes #882

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
Co-authored-by: Lois Wells <[email protected]>
Allows mobile users to save their project and remix in place of auto
remixing (#843)
Minor updates for project site integration
closes #733

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Magdalena Jadach <[email protected]>
Co-authored-by: Scott <[email protected]>
Co-authored-by: Scott Adams <[email protected]>
closes #889

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Scott <[email protected]>
… 3.11.1 to 4.0.0 (#895)

We need to be able to conditionally set headers for pyodideWorker.js in
pull request #893
but this feature isn’t supported in webpack-dev-server 3.x.x. We
therefore need to upgrade to version 4.0.0 which allows a function to be
provided in the headers option. When attempting this upgrade, I ran into
[this issue](storybookjs/storybook#22431) and
only managed to resolve it by also updating yarn to version 3.

I’ve tried to follow the migration guide here:

https://github.com/webpack/webpack-dev-server/blob/master/migration-v4.md
closes #897

---------

Co-authored-by: create-issue-branch[bot] <53036503+create-issue-branch[bot]@users.noreply.github.com>
Co-authored-by: Lois Wells <[email protected]>
## [0.21.2] - 2024-01-23

### Changed

- Minor copy changes to HTML add file modal
- Toggle errors sent via apiCallHandler off (#890)
- Upgrade webpack-dev-server to 4.0.0 to support conditional headers
- Upgrade yarn to 3.4.1 to workaround a string-width issue

### Fixed

- Editor input not focussing on iPad (#898)
Yarn v3 needs:
- activating inside Dockerfile
- volumes need mounting in docker-compose

### Notes

- `yarnrc.yml` could be different between environments (i.e. local
machine and container) so ignoring the file in the repo and generating
one on build in docker
Copy link

CHANGELOG.md Outdated
Comment on lines 13 to 26
## [0.21.2] - 2024-01-23

### Changed

- Minor copy changes to HTML add file modal
- Toggle errors sent via apiCallHandler off (#890)
- Upgrade webpack-dev-server to 4.0.0 to support conditional headers
- Upgrade yarn to 3.4.1 to workaround a string-width issue

### Fixed

- Editor input not focussing on iPad (#898)

## [0.21.1] - 2024-01-11
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This bit's here twice (see below)

@sra405 sra405 temporarily deployed to previews/update-output-only-branch January 26, 2024 12:40 — with GitHub Actions Inactive
Copy link

Copy link
Contributor

@loiswells97 loiswells97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sra405 sra405 merged commit a320956 into output-only-wc Jan 26, 2024
@sra405 sra405 deleted the update-output-only-branch branch January 26, 2024 13:35
@sra405 sra405 restored the update-output-only-branch branch January 26, 2024 13:36
Copy link

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