Skip to content
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

[Content Author Module 4: Images] [ED-med] Introduction seems tautological #661

Open
iadawn opened this issue Sep 20, 2022 · 2 comments
Open
Labels
done Done. Will close when related PRs are merged.

Comments

@iadawn
Copy link

iadawn commented Sep 20, 2022

The introduction to the topic 'Informative Images' is:

Informative images convey information visually, for example through a picture or illustration.

This seems to say 'informative images are images with information' rather than give some good examples. For example:

Informative images convey information visually, for example a graph or a floorplan

Or something like that

@daniel-montalvo
Copy link
Collaborator

Thanks @iadawn

I changed the sentence structure so that we clearly communicate that what we are listing there are occurrences of informative images.

I changed "picture" to "photo" to help clarifying the examples.

Informative images convey information visually. Examples include icons, photos, graphs, and illustrations. [...]

It is difficult for me to think of a floor plan as an informative image. I would feel more comfortable not including that at this stage. Would you mind if we leave it out given that we'd have now more concrete examples?

@daniel-montalvo daniel-montalvo added the question Further information is requested label Sep 21, 2022
@iadawn
Copy link
Author

iadawn commented Sep 22, 2022

The changes are fine.

My concern was more it was roughly saying that examples of informative were images.

@daniel-montalvo daniel-montalvo added done Done. Will close when related PRs are merged. and removed question Further information is requested labels Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Done. Will close when related PRs are merged.
Projects
None yet
Development

No branches or pull requests

2 participants