-
Notifications
You must be signed in to change notification settings - Fork 9
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 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
Co-authored-by: MagdalenaJadach <[email protected]>
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:  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
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)
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
|
loiswells97
reviewed
Jan 26, 2024
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 |
There was a problem hiding this comment.
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)
|
loiswells97
approved these changes
Jan 26, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.