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

Stop burying the lead #29306

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Stop burying the lead #29306

merged 1 commit into from
Feb 8, 2024

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Feb 7, 2024

Overview

Re-order field display on ManageEvent Info screen

I've been configuring a few event forms & find it kinda hard to find the most important fields - which I've kinda defined as the required fields - in particular the Event Title feels to me like the MOST important field - and it is number 5 after 'Participant Listing'

Also the start & end dates are important to have up front because start date is a required field & cos they are so easily misconfigured & the misconfiguration can block registrations that putting them in front of people's eyes seems helpful

Before

image

After

image

Technical Details

Comments

I was tempted to bring up max participants too - but then that can lead into waitlist & such like

Copy link

civibot bot commented Feb 7, 2024

🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷

Introduction for new contributors...
  • If this is your first PR, an admin will greenlight automated testing with the command ok to test or add to whitelist.
  • A series of tests will automatically run. You can see the results at the bottom of this page (if there are any problems, it will include a link to see what went wrong).
  • A demo site will be built where anyone can try out a version of CiviCRM that includes your changes.
  • If this process needs to be repeated, an admin will issue the command test this please to rerun tests and build a new demo site.
  • Before this PR can be merged, it needs to be reviewed. Please keep in mind that reviewers are volunteers, and their response time can vary from a few hours to a few weeks depending on their availability and their knowledge of this particular part of CiviCRM.
  • A great way to speed up this process is to "trade reviews" with someone - find an open PR that you feel able to review, and leave a comment like "I'm reviewing this now, could you please review mine?" (include a link to yours). You don't have to wait for a response to get started (and you don't have to stop at one!) the more you review, the faster this process goes for everyone 😄
  • To ensure that you are credited properly in the final release notes, please add yourself to contributor-key.yml
  • For more information about contributing, see CONTRIBUTING.md.
Quick links for reviewers...

➡️ Online demo of this PR 🔗

@civibot civibot bot added the master label Feb 7, 2024
@aydun
Copy link
Contributor

aydun commented Feb 8, 2024

Makes far more sense. Why didn't anyone do this years ago?!

@colemanw colemanw merged commit 934434f into civicrm:master Feb 8, 2024
3 checks passed
@colemanw colemanw deleted the info branch February 8, 2024 01:20
@eileenmcnaughton
Copy link
Contributor Author

@aydun that is an interesting question

@totten
Copy link
Member

totten commented Feb 8, 2024

Makes far more sense. Why didn't anyone do this years ago?!

At some point, we all decided that HTML was scary.

@colemanw
Copy link
Member

colemanw commented Feb 8, 2024

I think there are probably a lot more “easy wins” like this that user-feedback could generate. We just need a better communication channel for gathering user-feedback.

@TychoSchottelius
Copy link

I have some user feedback for you here:
the changes in the order make total sense except for one thing that confuses a large organization in our company:
if there are event templates, the selection field for them is now displayed in fourth place. If you work through the form from top to bottom, selecting the template overwrites or deletes several entries that you have previously made.
Before, the templates, if there were any, were displayed at the top, which was quite useful for processing.
if you, Eileen, could imagine restoring this, I can have it financed.

@totten
Copy link
Member

totten commented Sep 24, 2024

Yes! I've had the same confusion. If the "Template" is going to reset everything (which is perfectly reasonable), then it needs to come first.

@eileenmcnaughton
Copy link
Contributor Author

@TychoSchottelius are you suggesting you would put up a patch to move the template to the top? Seems OK if that is what you are suggesting

@TychoSchottelius
Copy link

Yes, Eileen, if think so. I will pass this to Dominic and he will come back with a batch for this.
thank you for the quick response!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants