-
Notifications
You must be signed in to change notification settings - Fork 53
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
Website Package version number is illegible due to the text colour #2360
Comments
I can't remember...was it ever readable...I feel like it was |
yep, i had an old screenshot in some training slides and it used to be. |
I believe it just happened when going from 1.0.1 to 1.0.2? |
not sure sorry! |
It must be default colour, but it seems feasible to change it. This could be defined within the development section of the _pkgdown.yml file. |
@Fanny-Gautier this seems a good article: https://pkgdown.r-lib.org/articles/customise.html. Would you be interested in self-assigning this one and having an explore to see if you can get it looking cleaner? |
thanks Fanny! the default you show there was how it used to be and that light grey colour was fine but for some reason it now went to dark grey which conflicts with the background. light grey or white would be best as the other colours look confusing, like the red one would make me assume the package is not ready. |
I am well getting the version number in black with the release mode with the below setting: We can force it with the light grey color by setting the version_label to "default" as per below statements: Could you please confirm that I should force it to "default" in the _pkgdown.yml file, whatever the mode (auto/release) ? It means that the version number does not appear in red anymore when we work on development branches. Current _pkgdown.yml file: Otherwise it needs to be updated for the official release only, when the mode is set to release. |
Ideally, it would be red for the dev site, and then grey/white for the main site (this is how it was before) |
@ddsjoberg, @cicdguy @dgrassellyb - can you all weigh in here. something happened since 1.0.0 to 1.0.1/2 where our package version is hard to read now. |
Is _pkgdown.yml file updated and the current mode updated from "auto" to "release" for the official release ? What is the process for the official releases ? |
Here is some rough guidance on how we do it. https://pharmaverse.github.io/admiraldev/dev/articles/release_strategy.html You would need to inspect the CI action for pkgdown as well in the admiralci repo |
@cicdguy, could you please check if this issue get resolved while fixing the "404 errors - previous versions of admiral website" ? |
@Fanny-Gautier - as indicated in yesterday's call, it's on track for resolution following the implementation of the issues mentioned here, reverting the workflows to when the multiversion pkgdown docs were working (before then introduction of the custom solution that @ddsjoberg has implemented), and restoring the previous versions from the git history. |
Oh sorry I misunderstood @Fanny-Gautier - I don't believe fixing the missing versions issue will actually fix the underlying issue we're witnessing here (discoloration of the version). That will in fact be fixed by what you already mentioned eariler:
|
I guess this is confusing as the version number used to be a different color and has only recently changed to be unreadable. can we hold off on this @Fanny-Gautier until multi-versions is re-implemented please. |
@cicdguy to be clear, all updates to admiral workflows were done by the IDR team. I added an article to the pkgdown site that links to the websites. |
@ddsjoberg - okay let's take down this article then. It's been a source of confusion since its inception. |
there's no need to modify the plans we made in our meeting yesterday |
Indeed. No intention of modifying it - it's still part of the plan. We'll be removing the article once the new features have been implemented, given that it'll be part of the reversion to the old state. |
Is there any update for this topic or is still on hold ? Thanks. |
@cicdguy said there is progress on it and should be available mid-Mayish. Thanks for checking!! |
I've restored the previous pages. Will be updating the workflows soon. |
Thanks @cicdguy !! I noticed in the dev mode of the site that the dropdowns have turned dark. Not sure if was due to the update? |
OR it could be the default in the |
So @manciniedoardo and I looked into the dark background issue and it looks like the |
Thanks all - I've been paying attention to color stuff more since the version number went dark on us 6 months ago. I couldn't remember if this dark menu had always been present or not. Thanks for checking. |
I am working on it today. But I find difficult to understand which --bs- variable is used in the CSS file when the |
@cicdguy should we be going down this path? Is it possible for the CI action for the pkgdown build to take care of this type of thing for us? I don't remember us messing with the stuff when the pkg version was a white color so unclear when or why this dark color started to appear. |
Yeah I have no idea. I'm not a front end expert so my understanding of the issue is very limited 😢 |
I think this is coming from this line in the CSS with <small class="nav-text text-muted me-auto" data-bs-toggle="tooltip" data-bs-placement="bottom" title="Released version">1.0.2</small> The tricky part will be overriding the default. A couple of things I would try (and apologies if this is already suggested above, it didn't read the whole thing 🤷🏼 )
|
Yesterday, I saw a previous post from you on a forum and tried to search for that line, but I could not find it in the CSS file. Did I look at the wrong place? |
Shall we just give it a try now? |
… mode, and keep red (danger) for auto/development mode
I have created the feature branch and pushed the updates. Shall I create a PR or is it fine for you to try from the corresponding feature branch? |
Could you branch off the latest release? I wanted to re-release admiral to try out the fix, but not have any of the new stuff visible. https://stackoverflow.com/questions/10940981/how-to-create-a-new-branch-from-a-tag |
The plot thickens! |
@bms63 I have created 2360_website_package_version_number_color_tag_v1-0-2 as per your suggestion. I thought that it was due to the other statements commented out (#mode: release + #version_label: success) so I deleted them, but it does not matter. still switching to green now on other pages than the index one 😵😵😵 |
that is interesting behavior...but i don't really mind it if the version label goes back to white. |
I have restarted R, and will run build_site() instead of build_articles()... I keep you in touch once it's ready for you to test an official release. |
Hi @bms63 , |
@Fanny-Gautier @bms63 It's running fine for me on branch 2360_website_package_version_number_color_tag_v1-0-2. |
@bms63 Is it possible that I face the errors due to the admiraldev package version ? Lina is having 1.0.0.9011 while I have 1.0.0.9027. It seems that my error comes from an update in admiraldev between version 1.0.0.9016 and 1.0.0.9017. thanks |
Okay! It finally made sense to me why we needed to install v1.0.0 for admiraldev...this is admiral 1.0.2 It looks fine to me. Can we change it to the light gray and just merge to main from a non-tagged branch. I'm not going to have time to test it out with a mini-release...will just do it live!! :) |
Background Information
Definition of Done
No response
The text was updated successfully, but these errors were encountered: