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

documentation: labs: kernel api: improve quality of list_evolution.png #406

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hanyujie2002
Copy link

@hanyujie2002 hanyujie2002 commented May 10, 2024

The original image is too blurry to see clearly. So I draw a new one with draw.io. What's more, the new one is smaller in size after compressing

@hanyujie2002
Copy link
Author

This should works fine :)

@lkt-bot
Copy link
Collaborator

lkt-bot commented May 10, 2024

@hanyujie2002
Copy link
Author

The document reference this picture using code below:

.. image:: ../res/list_evolution.png
   :width: 85%

I recommend changing it to

.. image:: ../res/list_evolution.png
   :width: 1000px

to improve the responsibility of this picture.

I would like to hear your opinion. With your permission, I will create a new commit for that. @dbaluta

@hanyujie2002
Copy link
Author

hanyujie2002 commented May 10, 2024

The document reference this picture using code below:

.. image:: ../res/list_evolution.png
   :width: 85%

I recommend changing it to

.. image:: ../res/list_evolution.png
   :width: 1000px

to improve the responsibility of this picture.

I would like to hear your opinion. With your permission, I will create a new commit for that. @dbaluta

I have already apply that to the Chinese localization. See https://linux-kernel-labs-zh.xyz/labs/kernel_api.html

You can evaluate the effect in that link

What's more, the format used in the Chinese localization is svg. With your permission, I will also submit a new commit changing the picture format.

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.

2 participants