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

REF Simple cleanup of tabset code for contributionpages #14616

Merged

Conversation

mattwire
Copy link
Contributor

Overview

Just simplify the code a little bit (per Event TabHeader.php). There is a lot of further scope for consolidating the two classes.

Before

Duplicate values in array

After

Shared values added to each array element

Technical Details

Comments

@civibot
Copy link

civibot bot commented Jun 23, 2019

(Standard links)

@civibot civibot bot added the master label Jun 23, 2019
@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

@mattwire other than the style warnings this is fine - ie. it causes no regression & makes sense. I did wonder if we even need to set those falsey-defaults and I tried just removing the + $defaults and got no regression or enotices so we could possibly also leave off. Either way if you can sell jenkins on this I'm good with it.

@mattwire mattwire force-pushed the contributionpage_tabset_cleanup branch from a0da843 to a2d73a9 Compare July 4, 2019 13:46
@mattwire mattwire force-pushed the contributionpage_tabset_cleanup branch from a2d73a9 to a8b477f Compare July 4, 2019 13:59
@mattwire
Copy link
Contributor Author

mattwire commented Jul 7, 2019

@eileenmcnaughton Do you want to merge?

@eileenmcnaughton eileenmcnaughton merged commit c5295cd into civicrm:master Jul 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants