darxtrix/ptop

Separate out the hardcoding from GUI.py

Open

#20 opened on Jul 12, 2017

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Python (573 stars) (43 forks)batch import
good first issuerefactoring

Description

All of the positions, lengths, widths are hardcoded. Need to separate them in a different config file or in the same file.

Take into account the situations in which the Terminal screen is quite small. The following strategy can be used.

  • Ptop can try to be responsive till a particular stage
  • After that the user can do scrolling to see the full view.

Contributor guide

Tech stack
python
Domain
cli
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Pythonbasic file I/O
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75
Research direction
First, examine GUI.py to identify all hardcoded positions, lengths, and widths. Then design a configuration file format (e.g., JSON or YAML) to store these values. Consider the responsive behavior for small terminal screens mentioned in the issue: ptop should be responsive up to a point and then allow scrolling. Look at any existing comments or linked resources for more context. Implement the extraction and loading of config values, ensuring the GUI reads from the config file at startup.