Skip to content
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

Adjust the symbols list #6339

Closed
wants to merge 3 commits into from
Closed

Adjust the symbols list #6339

wants to merge 3 commits into from

Conversation

feerrenrut
Copy link
Contributor

Fixes #6101
Extends PR #6178

Put the 'symbol' label above the list and have the list itself take up the full width of the dialog. Columns are spaced automatically, with one nominated to take up any extra available room.

The first column was selected to expand more, as the information in the other columns can be viewed in the "change symbol" group.

nishimotz and others added 2 commits July 15, 2016 16:43
…take up the full width of the dialog. Column items should have the same width, rather than specifying the widths by pixels.
This custom control extends a listCtrl with the help of a mixin so that
a column can be specified to be resized to take up the remaining width
available to a listCtrl.
@feerrenrut
Copy link
Contributor Author

@jcsteh Would you mind reviewing this?

@feerrenrut feerrenrut assigned feerrenrut and jcsteh and unassigned feerrenrut Sep 6, 2016
# -*- coding: UTF-8 -*-
#settingsDialogs.py
#A part of NonVisual Desktop Access (NVDA)
#Copyright (C) 2006-2016 NV Access Limited
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change file name above and copyright year to just 2016.

@feerrenrut
Copy link
Contributor Author

Accidentally referenced the wrong PR in the commit message. See: commit e533369 for incubating commit

feerrenrut added a commit that referenced this pull request Sep 29, 2016
Issue: #6101
Extends PR: #6178

Merge branch 'i6101_SymbolsListCtrl' into next

Conflicts:
	source/gui/settingsDialogs.py
@nvaccessAuto nvaccessAuto assigned feerrenrut and unassigned jcsteh Sep 29, 2016
feerrenrut added a commit that referenced this pull request Oct 13, 2016
Merges in branches
- 'i6101_SymbolsListCtrl'
- 'i6348-documentFormattingDialog'
- 'fixGuiAlignmentIssues'

Adjust the symbols list. See PR #6339

    Put the 'symbol' label above the list and have the list itself take up
    the full width of the dialog. Columns are spaced automatically, with one
    nominated to take up any extra available room.

    The first column was selected to expand more, as the information in the
    other columns can be viewed in the "change symbol" group.

    Fixes #6101

Restructure the document formatting dialog. See PR #6402

    The document formatting settings dialog is restructured so that items
    can no longer "disappear" off the screen. The options are now in a
    scrolling section, grouped for clarity (paritcularly for new users /
    sighted users). A dialog description has been added, for further
    clarity.

    Fixes #6348

Fix Gui Alignment issues. See PR #6287

    Fixes up various alignment and padding problems in the GUI.
    Introduces a new helper to abstract some of these details.

    Fixes #6317 (Gestures treeview does not look like a treeview)
    Fixes #5548 (Text control to specify the path when creating a portable copy is quite narrow)
    Fixes #6342 (Spacing between labels and combo boxes)
    Fixes #6343 (Inconsistency with vertical spacing between checkbox/label options)
    Fixes #6349 (input gestures dialog tree view too small)
@feerrenrut
Copy link
Contributor Author

This has been merged into master with commit: 87ed4d1

@feerrenrut feerrenrut closed this Oct 13, 2016
feerrenrut added a commit that referenced this pull request Oct 13, 2016
For PR #6287 - Various padding and alignment issues have been resolved. (#6317, #5548, #6342, #6343, #6349)
For PR #6402 - The document formatting dialog has been adjusted so that the contents scrolls. (#6348)
For PR #6339 - Adjusted the layout of the symbols pronunciation dialog so the full width of the dialog is used for the symbols list. (#6101)
@feerrenrut feerrenrut added this to the 2016.4 milestone Oct 13, 2016
@feerrenrut feerrenrut deleted the i6101_SymbolsListCtrl branch January 20, 2020 09:19
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.

None yet

3 participants