Skip to content

Add more modal customizations#347

Merged
martin-g merged 1 commit intoapache:masterfrom
andruhon:customise-modal
Mar 5, 2019
Merged

Add more modal customizations#347
martin-g merged 1 commit intoapache:masterfrom
andruhon:customise-modal

Conversation

@andruhon
Copy link
Contributor

More Modal settings as promised in discussion of #294 . Sorry it took longer than I expected, was buried in work.

@martin-g
Copy link
Member

For completeness just to mention that these settings could be set by overriding ModalWindow#postProcessSettings(JSONObject settings)

@andreikondratev
Copy link
Contributor

Hi! Is this change got stuck? Should I do anything to proceed?

@martin-g
Copy link
Member

martin-g commented Mar 5, 2019

I will create a ticket in JIRA for this PR and merge it.

@martin-g
Copy link
Member

martin-g commented Mar 5, 2019

@martin-g martin-g merged commit edf01e8 into apache:master Mar 5, 2019
martin-g pushed a commit that referenced this pull request Mar 5, 2019
WICKET-6640 Add settings for customizing the ModalWindow's spacing, header height and overflow

(cherry picked from commit edf01e8)
@martin-g
Copy link
Member

martin-g commented Mar 5, 2019

Thank you, @andruhon !

@andreikondratev
Copy link
Contributor

Should I next time create a jira ticket for change like this?

@solomax
Copy link
Contributor

solomax commented Mar 7, 2019

@andreikondratev JIRA tickets helps to create release notes
So, I believe, it will speed up things

@martin-g
Copy link
Member

martin-g commented Mar 7, 2019

Yes, most users don't read the git commit history to see what's new in a release. Wicket distributions (.tar.gz and .zip) contain RELEASE_NOTES file with the export from JIRA.

GitHub releases (actually Git tags) are not official Apache releases.

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.

4 participants