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

Improved milestone selection dialog #741

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Tunous
Copy link
Collaborator

@Tunous Tunous commented Sep 23, 2017

I've changed the milestone selection dialog to consist of view pager that allows for selection of closed milestones.

I'm not 100% sure about the look of this. We could modify a bit the appearance of milestones to highlight which one is selected. And I'm not really sure if the "deselect" button is a good idea. Sadly I don't have a better one. Also, we might want to colorize the pager tabs.

What is missing is the functionality of old "Manage milestones" buttons. We could add it like in old dialog (perhaps stack buttons) but I'm open to other suggestions.

Preview:
milestone-dialog

Closes #570

@Tunous Tunous added the wip label Sep 23, 2017
@maniac103
Copy link
Collaborator

I'm not 100% sure about the look of this. We could modify a bit the appearance of milestones to highlight which one is selected.

That would indeed make sense, I think.

And I'm not really sure if the "deselect" button is a good idea. Sadly I don't have a better one.

What about placing a full-width 'Select none' or 'Any milestone' (wording improvements welcome) between the view pager and the button bar? That way we'd not mix selection and dialog dismissal in the button bar.

Also, we might want to colorize the pager tabs.

Worth a try, but my gut feeling is that the current version will look better. In general I think having more than one large block of color (let alone different colors) looks 'too much'. It's basically the same thing as with the advanced editor.

What is missing is the functionality of old "Manage milestones" buttons. We could add it like in old dialog (perhaps stack buttons) but I'm open to other suggestions.

What do you mean by that?

@Tunous
Copy link
Collaborator Author

Tunous commented Sep 26, 2017

What is missing is the functionality of old "Manage milestones" buttons. We could add it like in old dialog (perhaps stack buttons) but I'm open to other suggestions.

What do you mean by that?

Go to issue edit screen and click on the button to add milestone. You can see here that the dialog has additional button which is missing in new design.

@maniac103
Copy link
Collaborator

Go to issue edit screen and click on the button to add milestone. You can see here that the dialog has additional button which is missing in new design.

Ah, right. Should be easy to add though when moving the deselect button out.

@Tunous Tunous self-assigned this Sep 26, 2017
@Tunous Tunous removed the wip label Sep 27, 2017
@Tunous
Copy link
Collaborator Author

Tunous commented Feb 3, 2019

Rebased onto latest master and finished missing changes. I'm still not convinced about the looks of this but at least now everything is functional.

@Tunous Tunous removed their assignment Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Assign closed milestone to issue/pr
2 participants