Skip to content

Refactor: filesPage.py changed QtWidgets.QListWidgetItem to our custom EntryListModel#111

Closed
gmmcosta15 wants to merge 6 commits intodevfrom
refactor/files_page_window
Closed

Refactor: filesPage.py changed QtWidgets.QListWidgetItem to our custom EntryListModel#111
gmmcosta15 wants to merge 6 commits intodevfrom
refactor/files_page_window

Conversation

@gmmcosta15
Copy link
Collaborator

@gmmcosta15 gmmcosta15 commented Dec 2, 2025

What Changed?

helper_methods.py: created method get_file_name that accepts a string with the filepath and returns only the filename
filesPage.py: Changed implementation of how the files are showed, from a ´QtWidgets.QListWidgetItem´ to our custom EntryListModel

Screenshots

  • Visually, UI is unchanged

Future Work

  • Implementation of the delete file with the on_delete_file Slot
  • Unit and Integration Tests

@gmmcosta15 gmmcosta15 changed the title Refactor/files page window Refactor: filesPage window, changed QtWidgets.QListWidgetItem´ to our custom EntryListModel Dec 2, 2025
@gmmcosta15 gmmcosta15 self-assigned this Dec 2, 2025
@gmmcosta15 gmmcosta15 requested a review from HugoCLSC December 2, 2025 16:21
@gmmcosta15 gmmcosta15 added the Refactor Enhancing code's readability, maintainability, and extensibility while addressing technical debt. label Dec 2, 2025
@gmmcosta15 gmmcosta15 changed the title Refactor: filesPage window, changed QtWidgets.QListWidgetItem´ to our custom EntryListModel Refactor: filesPage.py changed QtWidgets.QListWidgetItem to our custom EntryListModel Dec 2, 2025
@gmmcosta15 gmmcosta15 changed the title Refactor: filesPage.py changed QtWidgets.QListWidgetItem to our custom EntryListModel Refactor: filesPage.py changed QtWidgets.QListWidgetItem to our custom EntryListModel Dec 2, 2025
@gmmcosta15 gmmcosta15 marked this pull request as ready for review December 2, 2025 16:23
Guilherme Costa added 5 commits December 11, 2025 11:33
…itemclick

helper_methods: updated the get_file_loc method to return always the filename instead of the full path

Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
…file_name

Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
…g commented lines

Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
@gmmcosta15 gmmcosta15 force-pushed the refactor/files_page_window branch from e8b88d5 to 215a8ca Compare December 11, 2025 11:35
Signed-off-by: Guilherme Costa <guiherme.costa@blockstec.com>
@gmmcosta15 gmmcosta15 closed this Dec 11, 2025
@gmmcosta15 gmmcosta15 deleted the refactor/files_page_window branch December 12, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor Enhancing code's readability, maintainability, and extensibility while addressing technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant