-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC: whats new other
section reduced
#40900
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
Conversation
attack68
commented
Apr 12, 2021
- closes DOC: move *most* of Other Bug Fix section #40897
doc/source/whatsnew/v1.3.0.rst
Outdated
@@ -117,6 +117,9 @@ We provided some focused development on :class:`.Styler`, including altering met | |||
to accept more universal CSS language for arguments, such as ``'color:red;'`` instead of | |||
``[('color', 'red')]`` (:issue:`39564`). This is also added to the built-in methods | |||
to allow custom CSS highlighting instead of default background coloring (:issue:`40242`). | |||
Enhancements to other built-in methods include extending the :meth:`.Styler.background_gradient` | |||
method to shade elements based on a given gradient map and not be restricted only to | |||
values in the DataFrame (:issue:`39930`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
emm, just one nitpick, is 39930
an issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nah, i don't think there is sphinx for prs, if so, would suggest to use (:issue:`22727`, :issue:`28901`)
, instead of (:issue:`39930`)
. Since usually put issue numbers to track in whatsnew, the 39930 is not issue but pr, then users who check it might get confused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no they are the same (issue or pr), but yeah you can list all 3 (we usually list just the issues, however if the PR adds somehing non-trivial then can list that 2)
thanks @attack68 |