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

Show conflicts in changeset #3727

Merged

Conversation

HebaruSan
Copy link
Member

@HebaruSan HebaruSan commented Nov 27, 2022

Background

When a user tries to install mods that conflict with one another or with an installed mod, the apply changes button is disabled and a description of the conflict appears in the leftward part of the status bar.

Motivation

It has been fairly common recently for users to ask why the apply changes button is disabled; many or most users don't notice the text in the lower left until it's pointed out.

Changes

Now when the changeset has conflicts, you can still click the apply changes button, but the conflicting mods are highlighted in red in the changeset, the conflict reasons take over the description column, and the apply button is disabled:

image

Hopefully this will make it more discoverable.

@HebaruSan HebaruSan added Enhancement New features or functionality Easy This is easy to fix GUI Issues affecting the interactive GUI Pull request labels Nov 27, 2022
@HebaruSan HebaruSan force-pushed the feature/apply-disabled-tooltip branch from a5fe268 to 5d335bb Compare November 27, 2022 00:52
@vinix38
Copy link
Contributor

vinix38 commented Nov 27, 2022

I have a suggestion: if there are conflicts, clicking the apply button will bring a popup describing the conflicts that are present and why changes cannot be applied.
I don't know if that would be useful or disruptive, what do you think?

@HebaruSan

This comment was marked as outdated.

@HebaruSan HebaruSan force-pushed the feature/apply-disabled-tooltip branch from 5d335bb to 2fc0dbb Compare February 27, 2023 03:15
@HebaruSan

This comment was marked as resolved.

@HebaruSan HebaruSan changed the title Show apply changes disable reason in tooltip Show conflicts in changeset Feb 27, 2023
Copy link
Member

@techman83 techman83 left a comment

Choose a reason for hiding this comment

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

Looks great! Making things clearer is always a great thing!

@HebaruSan HebaruSan merged commit b80461d into KSP-CKAN:master Mar 1, 2023
@HebaruSan HebaruSan deleted the feature/apply-disabled-tooltip branch March 1, 2023 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Easy This is easy to fix Enhancement New features or functionality GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants