Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • D developer-www
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 22
    • Issues 22
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 32
    • Merge requests 32
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Teams
  • DocumentationDocumentation
  • developer-www
  • Merge requests
  • !117

Review beginner tutorial for Python

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Arjan Molenaar requested to merge amolenaar/developer-www:python-tutorial-review into main Dec 18, 2022
  • Overview 9
  • Commits 3
  • Pipelines 2
  • Changes 6

Reviewed the Beginner Tutorial Python code.

I used GNOME Builder 43.

While reviewing I assumed the tutorial is performed front-to-back.

  • Removed GSettings setup stuff from dark_mode page, since that was already done a few pages before
  • Cleaned up open_file_complete on toaster page, to be similar to what we've built in opening_files
  • Fixed a bunch of emphasized lines
  • Few textual changes
  • Examples have been updated to reflect the GNOME Builder 43 project template

Some things to improve still:

  • self._native is not needed for dialogs, I think
  • No GSettings files have been included in the Python project template
  • The Shortcuts UI file is not present in the Python project template
  • the display name code is defined in both the load and save code. It's probably better to extract that in its own method.

GNOME Builder bug:

  • When I name the project text-viewer the main class is name Text-viewerApplication.
Edited Dec 24, 2022 by Arjan Molenaar
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: python-tutorial-review