8000 Added tabTitle argument for iSEE by RiboRings · Pull Request #693 · iSEE/iSEE · GitHub
[go: up one dir, main page]

Skip to content

Added tabTitle argument for iSEE#693

Merged
federicomarini merged 1 commit intoiSEE:develfrom
RiboRings:tab_title
Apr 14, 2025
Merged

Added tabTitle argument for iSEE#693
federicomarini merged 1 commit intoiSEE:develfrom
RiboRings:tab_title

Conversation

@RiboRings
Copy link
Contributor

Hi!

As discussed in the miaDash submission to Bioc, the tab title for miaDash is currently derived from the dashboardHeader title and therefore is not rendered correctly when the title is an HTML element, like for miaDash.

Screenshot 2025-04-01 at 23 05 09

Therefore, I added a new argument to the iSEE function called tabTitle, which lets you modify the name of the browser tab without affecting the app title or other elements. For example, for the ISEE app:

Screenshot 2025-04-01 at 23 02 38

Feel free to let me know if this solution suits iSEE and if anything should be changed.

Best,
Giulio

@RiboRings RiboRings mentioned this pull request Apr 1, 2025
10 tasks
@federicomarini
Copy link
Contributor

I am guessing this is related to this #682 but not fully addressed, correct?
Happy to check it locally a bit, the win GHA is going nuts a bit but I think it is not really a problem

@RiboRings
Copy link
Contributor Author

Yes #682 added support for HTML elements as app title, but then the browser tab is not rendered neatly when using HTML elements as app title.

@federicomarini
Copy link
Contributor

We could have some alternative with respect to the default value of tabTitle, as it could simply be going as the appTitle itself, but for the sake of having this one in before the next release cycle, I am merging 😉

Thanks for the contribution @RiboRings !

@federicomarini federicomarini merged commit 21e1bdd into iSEE:devel Apr 14, 2025
2 of 3 checks passed
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

0