Skip to content

Fix typos in Doc/library/curses.rst#148994

Open
manoj-km24 wants to merge 5 commits intopython:mainfrom
manoj-km24:fix-curses-rst
Open

Fix typos in Doc/library/curses.rst#148994
manoj-km24 wants to merge 5 commits intopython:mainfrom
manoj-km24:fix-curses-rst

Conversation

@manoj-km24
Copy link
Copy Markdown
Contributor

@manoj-km24 manoj-km24 commented Apr 25, 2026

In Doc/library/curses.rst, the following sentence lacks a comma:

On software terminal emulators it will have a fixed high value

This PR adds a comma and improves readability and maintains grammar consistency.


📚 Documentation preview 📚: https://cpython-previews--148994.org.readthedocs.build/

Copy link
Copy Markdown
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix all typos in curses.rst if there are more?

@manoj-km24
Copy link
Copy Markdown
Contributor Author

Can you fix all typos in curses.rst if there are more?

I have fixed more typos, comma issues and made words as compound adjectives.

  • For 'upper left corner' like sentences , I changed it as 'upper-left corner' (as compound adjectives since corner is a noun)
  • For 'upper left hand corner' like sentences, I changed it as 'upper left-hand corner'

I could make this uniform if you wish (only upper left , no upper left hand) since it repeats in various places with different words like right ,lower etc.

@picnixz picnixz changed the title Add Comma for better readability in curses.rst Fix typos in curses.rst Apr 26, 2026
Comment thread Doc/library/curses.rst Outdated
Comment thread Doc/library/curses.rst Outdated
Comment thread Doc/library/curses.rst Outdated
Comment thread Doc/library/curses.rst Outdated
:ref:`window <curses-window-objects>` object in which the textbox is to
be contained. The edit cursor of the textbox is initially located at the
upper left hand corner of the containing window, with coordinates ``(0, 0)``.
upper left-hand corner of the containing window, with coordinates ``(0, 0)``.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can just say upper-left and remove the hand, can't we? @StanFromIreland

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
upper left-hand corner of the containing window, with coordinates ``(0, 0)``.
upper-left corner of the containing window, with coordinates ``(0, 0)``.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have made the requested changes.

@manoj-km24 manoj-km24 requested a review from picnixz April 27, 2026 14:30
@picnixz picnixz changed the title Fix typos in curses.rst Fix typos in Doc/library/curses.rst Apr 27, 2026
Copy link
Copy Markdown
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I only skimmed over it quickly.

Comment thread Doc/library/curses.rst Outdated
with :func:`newpad`. The additional parameters are needed to indicate what part
of the pad and screen are involved. *pminrow* and *pmincol* specify the upper
left-hand corner of the rectangle to be displayed in the pad. *sminrow*,
left-corner of the rectangle to be displayed in the pad. *sminrow*,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you meant "upper-left corner"?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you meant "upper-left corner"?

@StanFromIreland Thanks for pointing that out , I've fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants