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

Improve the About dialog #6552

Closed
jtpio opened this issue Oct 4, 2022 · 23 comments · Fixed by #6996
Closed

Improve the About dialog #6552

jtpio opened this issue Oct 4, 2022 · 23 comments · Fixed by #6996

Comments

@jtpio
Copy link
Member

jtpio commented Oct 4, 2022

The look of the current "About" dialog could be improved:

image

As a comparison this is what the about dialog in JupyterLab looks like:

image

Maybe the notebook dialog could be aligned to look more like the one in JupyterLab, and also display a link to the contributor list.

@jupyterlab-probot jupyterlab-probot bot added the status:Needs Triage Applied to issues that need triage label Oct 4, 2022
@jtpio jtpio added this to the 7.0 milestone Oct 4, 2022
@jtpio
Copy link
Member Author

jtpio commented Oct 4, 2022

This is where the About dialog is created:

const title = (
<>
<span className="jp-AboutNotebook-header">
<jupyterIcon.react width="196px" height="auto" />
</span>
</>
);
const notebookURL = 'https://github.com/jupyter/notebook';
const linkLabel = trans.__('JUPYTER NOTEBOOK ON GITHUB');
const externalLinks = (
<span>
<a
href={notebookURL}
target="_blank"
rel="noopener noreferrer"
className="jp-Button-flat jp-AboutNotebook-about-externalLinks"
>
{linkLabel}
</a>
</span>
);
const version = trans.__('Version: %1', app.version);
const body = (
<>
<span className="jp-AboutNotebook-body">{version}</span>
<div>{externalLinks}</div>
</>
);
const dialog = new Dialog({
title,
body,
buttons: [
Dialog.createButton({
label: trans.__('Dismiss'),
className:
'jp-AboutNotebook-about-button jp-mod-reject jp-mod-styled'
})
]
});
dialog.addClass('jp-AboutNotebook');
void dialog.launch();

@jtpio jtpio added help wanted good first issue and removed status:Needs Triage Applied to issues that need triage labels Oct 4, 2022
@krassowski
Copy link
Member

krassowski commented Oct 4, 2022

The Notebook v6 About dialog was even more informative:

Screenshot from 2022-10-04 10-20-01

In particular, showing jupyter-server version (and maybe also the Python version it is running on) might be worth considering since the server and frontend versions are now decoupled.

I think I remember that the kernel info was a separate menu item in JupyterLab at some point but I cannot find it any more - either a regression or I am wrong and it was never implemented.

@davinderpalrehal
Copy link

Is someone working on this issue? If not I would like to have a crack at it.

@jtpio
Copy link
Member Author

jtpio commented Oct 5, 2022

@davinderpalrehal feel free to pick it up and open a PR, even it's just a work-in-progress. Thanks!

@kostyafarber
Copy link
Contributor

@davinderpalrehal you still working on this?

@davinderpalrehal
Copy link

@kostyafarber plans are to work on this over the weekend, is it time sensitive? If it's needed urgently I don't mind if someone else picks it up, my intention was to have the PR open by Monday.

@kostyafarber
Copy link
Contributor

Oh no go ahead! Just wanted to make sure if you weren't working on it I could pick it up. Thanks.

@davinderpalrehal
Copy link

Sounds good, thanks @kostyafarber.

@davinderpalrehal
Copy link

Hey @kostyafarber , I couldn't finish setting up the repo on my laptop and get it up and running, if you are still free please proceed with picking this issue up.

@kostyafarber
Copy link
Contributor

No worries @davinderpalrehal I'll try check it out

@dohrisalim
Copy link

Is this issue still open? @kostyafarber @jtpio

@kostyafarber
Copy link
Contributor

If you think there are further improvements to be made and the maintainers agree, please go ahead!

@jtpio
Copy link
Member Author

jtpio commented Feb 8, 2023

We could probably do a quick pass on the CSS to style the links. This is what it looks like at the moment:

image

@Tony-j77
Copy link
Contributor

Tony-j77 commented Feb 19, 2023

Finishing this issue :)

@AdityaGirdhar
Copy link

@jtpio Do you have any more improvements in mind? Maybe adding some more information like Notebook v6 had (showing jupyter-server version, and also the Python version it is running on). I've a university break starting from next week and I'd love to contribute.

@krassowski
Copy link
Member

Of note the odd hover style seen on the screenshot in original issue is an upstream issue in JupyterLab. Contributions to solve: jupyterlab/jupyterlab#14084 are welcome!

@Aryasarkar008
Copy link

Aryasarkar008 commented Mar 9, 2023

hey is this issue still open?? is someone still working on it??

@yjrab
Copy link
Contributor

yjrab commented Jun 2, 2023

Is this issue still open? @jtpio
Any help needed regarding the CSS of the About dialog?

@andrii-i
Copy link
Contributor

andrii-i commented Jun 3, 2023

Hi @yjrab! Thank you for taking interest in fixing this issue. You are not required to ask for a permission to work on such issue, please feel free to open a pull request with proposed implementation.

@andrii-i
Copy link
Contributor

andrii-i commented Jun 3, 2023

Here is the css of "About" dialog in JupyterLab, can be used for reference: https://github.com/jupyterlab/jupyterlab/blob/e0b4de1eb37c9dd37c11e5319ed00cb8d37b8d92/packages/help-extension/style/base.css

@yjrab
Copy link
Contributor

yjrab commented Jun 3, 2023

Thank you @andrii-i for you reply.
I will check the reference and propose something accordingly.

@Razeeeeeeeeeee
Copy link

Hey @andrii-i,
Is this issue still open?
I'd like to work on it

@andrii-i
Copy link
Contributor

Hi @Razeeeeeeeeeee. Please feel free to work on it and open a PR when ready.

@jtpio jtpio linked a pull request Aug 3, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.