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

Datagrid: Calendar popup issue #6416

Closed
iamjamesreyracho opened this issue May 13, 2022 · 3 comments · Fixed by #6484
Closed

Datagrid: Calendar popup issue #6416

iamjamesreyracho opened this issue May 13, 2022 · 3 comments · Fixed by #6484
Assignees
Labels
type: demo-app bug 🪲 [2] Velocity rating (Fibonacci)

Comments

@iamjamesreyracho
Copy link
Contributor

Describe the bug
The calendar popup is unable to use in Win-Chrome and Win-Edge but it is working with Win-FF.

To Reproduce

Steps to reproduce the behavior:

  1. Go to https://main-enterprise.demo.design.infor.com/components/datagrid/example-editable.html
  2. Click an empty field in Order Date.
  3. Click the calendar icon.
  4. See error

Expected behavior
The calendar popup should be enabled.

Win-FF (New & Classic)
https://www.screencast.com/t/Au9HzW9v

Version

  • ids-enterprise: v4.64.0-dev

Screenshots
Win-Chrome & Win-Edge (New & Classic)
https://www.screencast.com/t/UGfNta8N

Platform

  • Browser Name: Chrome (windows)

Additional context
NA

@tmcconechy tmcconechy added type: regression bug ↩️ [2] Velocity rating (Fibonacci) labels May 13, 2022
@tmcconechy tmcconechy added this to To do in Enterprise 4.64.x (May 2022) Sprint via automation May 13, 2022
@davidcarlsonberg davidcarlsonberg self-assigned this May 18, 2022
@davidcarlsonberg davidcarlsonberg moved this from To do to In progress in Enterprise 4.64.x (May 2022) Sprint May 18, 2022
@davidcarlsonberg
Copy link
Contributor

davidcarlsonberg commented May 19, 2022

It seems to me this isn't actually a bug because the maxDate is set on those fields so it makes sense those days are disabled?

    columns.push({
        id: 'orderDate',
        name: 'Order Date',
        field: 'orderDate',
        formatter: Formatters.Date,
        editor: Editors.Date,
        editorOptions: {
            showLegend: true,
            legend: [{
                name: 'Weekend',
                color: '#579E95',
                dayOfWeek: [0,6]}],
                disable: {
                    years: [],
                    dates: [],
                    minDate: '01-01-2015',
                    maxDate: '01-01-2022',
                    dayOfWeek: [],
                    isEnable: false,
                    restrictMonths: false
                },
                showTime: false,
                showMonthYearPicker: true,
                placeholder: true
            }, validate: 'availableDate required date',
            width: 120
        }
    );

@tmcconechy
Copy link
Member

Yeah that makes sense @davidcarlsonberg - good catch - maybe we can just change the example dates so the range is more current so it works (for a while) or make those dates dynamiclly set to current month - 2 +2

@iamjamesreyracho
Copy link
Contributor Author

QA Passed. Moving this ticket to done. Thanks.
image

@iamjamesreyracho iamjamesreyracho moved this from Ready for QA (beta) to Done in Enterprise 4.64.x (May 2022) Sprint May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: demo-app bug 🪲 [2] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants