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

Fix: Updating window title if new file loaded #361

Conversation

BoolPurist
Copy link

@BoolPurist BoolPurist commented Sep 14, 2024

Fixes the following issue

Refactors: Encapsulated updating file watcher and window title into one function

Note: This was a regression bug.
In previous versions, this bug did not exit
See the following commit "a6bb495" with version 0.4.1

Reason why PR on branch v0.4

The current version on crates.io seems to use commit 588687e on this branch.
Therefore, the addressed bug comes up when installing inlyne via the following command

cargo install inlyne

It also seems that the most recent changes are done on this branch.

Fixes the following [issue](Inlyne-Project#360)

Refactors: Encapsulated updating file watcher and window title into one function

Note: This was a regression bug.
In previous versions this bug did not exit
See the following commit "a6bb495" with version 0.4.1
@CosmicHorrorDev CosmicHorrorDev added C-bug Category: Something isn't working C-released-regression Category: A regression that's already loose! labels Sep 14, 2024
@CosmicHorrorDev
Copy link
Collaborator

Would you mind switching the base branch to main? v0.4 only exists for backporting fixes and won't be used after the pending 0.5.0 release

@CosmicHorrorDev CosmicHorrorDev changed the base branch from v0.4 to main September 14, 2024 15:34
@CosmicHorrorDev
Copy link
Collaborator

Well that button didn't do what I wanted. I'll work on fixing the conflicts

@CosmicHorrorDev
Copy link
Collaborator

Ahhhh. I somehow forgot that this was already working as intended on main with

so this would strictly be for the v0.4 branch 😅. I could merge this as a backport, but I'm not sure if it's worth it or not since it'll also be fixed once 0.5.0 is released

@CosmicHorrorDev CosmicHorrorDev changed the base branch from main to v0.4 September 14, 2024 15:54
@BoolPurist
Copy link
Author

I think this path is then obsolete in that case. No biggie. Say the word and I close this PR.

@CosmicHorrorDev
Copy link
Collaborator

Fair enough. I'm sorry that I didn't realize earlier, thanks for your contribution nonetheless

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: Something isn't working C-released-regression Category: A regression that's already loose!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Name of window title bar is not updated after jumping to a different file via a local link
2 participants