Skip to content

[Editor] Allow the user to add and save an alt-text for images (bug 1844952) #16977

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 1 commit into from
Sep 19, 2023

Conversation

calixteman
Copy link
Contributor

No description provided.

@calixteman calixteman force-pushed the add_alt_text branch 2 times, most recently from fda7173 to 19575d1 Compare September 19, 2023 14:10
@calixteman
Copy link
Contributor Author

/botio-linux preview

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_preview from @calixteman received. Current queue size: 0

Live output at: http://54.241.84.105:8877/061189fce12d5cd/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/061189fce12d5cd/output.txt

Total script time: 1.46 mins

Published

@calixteman
Copy link
Contributor Author

/botio-linux preview

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Received

Command cmd_preview from @calixteman received. Current queue size: 0

Live output at: http://54.241.84.105:8877/00b2ab67eb04400/output.txt

@moz-tools-bot
Copy link
Collaborator

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/00b2ab67eb04400/output.txt

Total script time: 1.45 mins

Published

@Snuffleupagus
Copy link
Collaborator

There seem to be some problem with the dialog position in RTL-mode, since its right side is always aligned with the right edge of the browser window:

rtl

@Snuffleupagus
Copy link
Collaborator

Possibly better suited for a follow-up: Looking at the specification it appears that the text-color of the textarea should become "greyed out" when Mark as decorative active, and that doesn't seem to happen currently.

@Snuffleupagus
Copy link
Collaborator

Snuffleupagus commented Sep 19, 2023

There's potentially one (small) problem with positioning the dialog absolutely in the way that we're doing here, which is that if the browser window is resized to become smaller after the dialog has been opened (I happened to trigger this by opening the devtools) then part of the altText dialog will be invisible/inaccessible.

One possible, simple way, around that might be to pass in the eventBus to the AltTextManager and add a listener for "resize" on it and reset the dialog left/top values to auto if a resize event occurs while the dialog is open?

@calixteman
Copy link
Contributor Author

There seem to be some problem with the dialog position in RTL-mode, since its right side is always aligned with the right edge of the browser window:

rtl

It's strange: nothing happens if I change left in devtools but if use right instead it's ok.
I tried to write a basic test case but everything works fine.

Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

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

r=me, with one more comment; thank you!

@calixteman calixteman merged commit 54f6264 into mozilla:master Sep 19, 2023
@calixteman calixteman deleted the add_alt_text branch September 19, 2023 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants