Skip to content

bpo-39600, IDLE: Remove duplicated font names #18430

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

Merged
merged 1 commit into from
Feb 10, 2020
Merged

bpo-39600, IDLE: Remove duplicated font names #18430

merged 1 commit into from
Feb 10, 2020

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Feb 10, 2020

In the font configuration window, remove duplicated font names.

https://bugs.python.org/issue39600

In the font configuration window, remove duplicated font names.
@vstinner
Copy link
Member Author

Before, Cantarell is listed 4 times:

before

After, Cantarell is only shown once:

after

@codecov
Copy link

codecov bot commented Feb 10, 2020

Codecov Report

Merging #18430 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #18430     +/-   ##
=========================================
  Coverage   82.11%   82.12%             
=========================================
  Files        1955     1954      -1     
  Lines      588949   583604   -5345     
  Branches    44428    44428             
=========================================
- Hits       483624   479258   -4366     
+ Misses      95674    94700    -974     
+ Partials     9651     9646      -5     
Impacted Files Coverage Δ
Lib/distutils/tests/test_bdist_rpm.py 30.00% <0.00%> (-65.00%) ⬇️
Lib/distutils/command/bdist_rpm.py 7.63% <0.00%> (-56.88%) ⬇️
Lib/test/test_urllib2net.py 76.92% <0.00%> (-13.85%) ⬇️
Lib/test/test_smtpnet.py 78.57% <0.00%> (-7.15%) ⬇️
Lib/ftplib.py 63.85% <0.00%> (-6.06%) ⬇️
Lib/test/test_ftplib.py 87.11% <0.00%> (-4.72%) ⬇️
Tools/scripts/db2pickle.py 17.82% <0.00%> (-3.97%) ⬇️
Tools/scripts/pickle2db.py 16.98% <0.00%> (-3.78%) ⬇️
Lib/test/test_socket.py 71.94% <0.00%> (-3.77%) ⬇️
Lib/test/test_asyncio/test_base_events.py 91.84% <0.00%> (-3.30%) ⬇️
... and 321 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29b3fc0...b908a40. Read the comment docs.

@KenanHArik
Copy link

I'm at the PyCascades Sprint, and reviewed this PR. This looks good to me merge in.

@willingc
Copy link
Contributor

Thanks @KenanHArik and @hroncok for the reviews. Happy to merge.

@willingc willingc merged commit ed335cf into python:master Feb 10, 2020
@terryjreedy
Copy link
Member

Carol, I should have been allowed to review before this was merged.

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR, and @willingc for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR, and @willingc for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@terryjreedy
Copy link
Member

terryjreedy commented Feb 10, 2020

@Mariatta, @vstinner
The backport PR numbers
#18437 (3.8) tests ran ('status checks passed') but Azure Pipelines summary not passed.
#18436 (3.7) has a github build error.
bedevere also did not do its issue number, backport, news checks.
I suspect that these all prevent merging. Will re-request.

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR, and @willingc for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR, and @willingc for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-bot
Copy link

GH-18441 is a backport of this pull request to the 3.8 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-bot
Copy link

GH-18442 is a backport of this pull request to the 3.7 branch.

@python python deleted a comment from miss-islington Feb 10, 2020
@python python deleted a comment from miss-islington Feb 10, 2020
miss-islington added a commit that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
miss-islington added a commit that referenced this pull request Feb 10, 2020
In the font configuration window, remove duplicated font names.
(cherry picked from commit ed335cf)

Co-authored-by: Victor Stinner <vstinner@python.org>
@vstinner vstinner deleted the idle_fonts branch September 21, 2021 22:49
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.

8 participants