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

Optional Event Information and Simplified UI #29

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

rnunez80
Copy link

This version optionally displays event information, including the date, time and/or description. In addition, the UI of the variant block has been simplified by reducing the number of field sets to base plus 2 and making minor adjustments to field labels.

Event Details option and reduced the number of dropdowns
New entries for events fields and reduced complexity
events option added to readme
new features added
screenshot of simplified item options
image added
schema fieldsets imported from default to avoid issues in the search block plus will allow removing unnecessary code.
start = `${parsedDate.toLocaleString(
'en-US',
{hour: 'numeric', minute: 'numeric', hour12: true},
)}`;
Copy link
Member

@erral erral Oct 25, 2022

Choose a reason for hiding this comment

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

This should be done in some generic way, not to be attached to en-US locale always.

Copy link
Author

Choose a reason for hiding this comment

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

changed locales to 'default' instead of 'en-US'


if (item.end) {
const parsedDate = new Date(Date.parse(item.end));
end = ` - ${parsedDate.toLocaleString('en-US', {
Copy link
Member

Choose a reason for hiding this comment

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

idem.

Copy link
Author

Choose a reason for hiding this comment

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

changed locales to default instead of 'en-US'

CHANGELOG.md Outdated
- Simplified Block side menu to reduce the number of collapsible accordions
- Simplified fields labels
- Move Heading information to the same location as default variants

Copy link
Member

@erral erral Oct 25, 2022

Choose a reason for hiding this comment

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

do not modify the CHANGELOG file please, we handle this auth auto-changelog. Group your commits with prefixes like: "feat: Optional Event Date", "fix: simplified field labels", etc.

added srcSet and sizes to image to improve image size handleling by the browser
make time locale default instead of en-US
removed changes I made to change in my previous PM
search block with advanced options
@rnunez80
Copy link
Author

made a few changes including using srcSet for improved performance on slow internet and mobile devices

@erral
Copy link
Member

erral commented Oct 27, 2022

@libargutxi @ionlizarazu review these functional changes please.

@erral
Copy link
Member

erral commented Oct 27, 2022

I see many prettier related changes here, we should add our prettier config file to avoid such changes here.

@rnunez80
Copy link
Author

should I release this as a separate addon?

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.

None yet

2 participants