-
Notifications
You must be signed in to change notification settings - Fork 0
fix(ux): keep customer-facing editor guidance actionable #391
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
Closed
Closed
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
2f4bfc0
fix(ux): customer copy states the next action and hides internal boun…
seonghobae 0efb510
test(ux): cover every human-unit branch of the size guidance message
seonghobae ee78bb4
fix(ux): make the hostile non-Error image rejection fallback coverage…
seonghobae 5b8037d
fix(ux): single-line v8 ignore so the guarded fallback stays coverage…
seonghobae 1473210
fix(ux): use v8 ignore start/stop for the hostile-rejection fallback
seonghobae 594a0ae
fix(ux): export normalizeImageError and cover both arms with real tests
seonghobae 6c61f4f
fix(ux): reconcile customer copy ownership boundaries
seonghobae 427a172
test(image): preserve non-cloneable native errors
seonghobae 88d4044
test(image): codify fail-closed error metadata
seonghobae a00ece3
docs(image): make fail-closed error contract explicit
seonghobae af5575d
chore(ownership): return Base64Image source to canonical lane
seonghobae 0193966
chore(ownership): remove competing Base64Image test
seonghobae 307d645
chore(ownership): remove competing Base64Image hostile-error test
seonghobae 55bc494
chore(ownership): restore canonical image source-policy tests
seonghobae 3f26f69
chore(ownership): return toolbar copy to canonical lane
seonghobae 32bd33a
chore(ownership): return toolbar tests to canonical lane
seonghobae 6f40f2b
test(ux): require customer-facing image action copy
seonghobae 48e481c
fix(ux): remove implementation jargon from image action
seonghobae 20f9064
test: align toolbar keyboard queries with customer copy
seonghobae 3edd749
chore(ci): refresh required workflow evidence
seonghobae f3d6a6f
fix(ownership): leave SafeClipboard guidance with canonical owner
seonghobae File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| import { afterEach, describe, expect, it } from 'vitest'; | ||
| import { cleanup, render, screen } from '@testing-library/react'; | ||
| import { Editor } from '@tiptap/react'; | ||
| import { Toolbar } from './Toolbar.js'; | ||
| import { buildExtensions } from '../extensions/kit.js'; | ||
|
|
||
| let editor: Editor | undefined; | ||
|
|
||
| afterEach(() => { | ||
| cleanup(); | ||
| if (editor && !editor.isDestroyed) editor.destroy(); | ||
| editor = undefined; | ||
| }); | ||
|
|
||
| describe('Toolbar customer-facing copy', () => { | ||
| it('keeps implementation jargon out of the image action accessible name', () => { | ||
| const element = document.createElement('div'); | ||
| document.body.appendChild(element); | ||
| editor = new Editor({ | ||
| element, | ||
| extensions: buildExtensions({ image: { maxDimension: 0 } }), | ||
| content: '<p>hello</p>', | ||
| }); | ||
|
|
||
| render(<Toolbar editor={editor} />); | ||
|
|
||
| expect( | ||
| screen.getByRole('button', { name: 'Insert inline image' }), | ||
| ).toBeInTheDocument(); | ||
| expect( | ||
| screen.queryByRole('button', { name: /base64/i }), | ||
| ).not.toBeInTheDocument(); | ||
| }); | ||
| }); |
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.