Coder Social home page Coder Social logo

Comments (26)

barbalex avatar barbalex commented on May 16, 2024 8

Here is my solution: https://github.com/barbalex/apf2/blob/master/src/components/shared/DateFieldWithPicker.js

Basic idea:

  • material-ui DatePicker is hidden by sizing it 0x0 pixels
  • instead, a textfield is shown and next to it a calendar icon
  • when this icon is clicked, focus is set to the DatePicker
  • which opens the calendar

Because I use moment.js to parse the date string

moment(date, `DD.MM.YYYY`).format(`YYYY-MM-DD`):

users have a lot of possibilities:

  • use the date picker
  • write a date manually:
    • write only day number if date is in the current month - moment.js will add current month and year
    • write day and month number if date is in current year- moment.js will add year
    • write entire date but you don't need to use two digits for single digit numbers - moment.js will add them for you
    • write entire date with all digits if you really want to

You can watch it in action here:
datefield

from mui-x.

markau avatar markau commented on May 16, 2024 6

This is important for accessibility (WCAG Level A). Having the option to manually enter a date string would be fantastic. I saw another Material implementation (that I don't remember now!) where they had a text input field, with an icon that opened the date picker. Best of both worlds.

from mui-x.

nathanmarks avatar nathanmarks commented on May 16, 2024 5

@mbrookes @tintin1343

Can the DatePicker be used without the built in TextField? That would allow the user full control over the textbox and when/how to display the picker.

from mui-x.

flexdeep avatar flexdeep commented on May 16, 2024 1

Thank's to all, that's something that I've been looking for. I've simplified barbalex's component removing unnecessary (in my opinion) dependencies. That's the component that I'm currently using (material-ui ver 0.18.6) :

'use strict';
import React, {Component} from 'react';
import {DatePicker, IconButton, TextField} from "material-ui";
import ActionDateRange from 'material-ui/svg-icons/action/date-range';
import format from 'date-fns/format'
import parse from 'date-fns/parse'

export default class DatePickerField extends Component{

   constructor(props){
      super(props);

      this.state = {
        selectedDate: new Date(),
        dateText: format(new Date(), 'MM/DD/YYYY')
      };
   }

   handleChangeDatePicker = (event, date) => {
      this.setState({selectedDate: date, dateText:format(date, 'MM/DD/YYYY')});
   };

   handleDateInputChange = (event, value) => {
      this.setState({dateText:value});
   };

   handleDateInputBlur = (value) => {
     let parsedDate = parse(value, 'MM/DD/YYYY');

     if(this.isADate(parsedDate)){
        this.setState({selectedDate:parsedDate});
     }
     else{
        this.setState({dateText:format(this.state.selectedDate, 'MM/DD/YYYY')});
     }
   };

   isADate = (maybeDate) => {
      if ( Object.prototype.toString.call(maybeDate) === "[object Date]" ) {
         if ( isNaN( maybeDate.getTime() ) ) {
            return false;
         }
         else {
            return true;
         }
      }
      else {
         return false;
      }
   };

   render(){

      let dateInputWidth = "150px";
      let datePickerMargin = "-185px";

      return (
         <div style={{display: "flex"}}>
            <TextField
               style={{width:dateInputWidth}}
               value={this.state.dateText}
               onChange={this.handleDateInputChange}
               onBlur={(event) => this.handleDateInputBlur(event.currentTarget.value)}
            />

            <IconButton style={{opacity:"0.65"}}
                        onClick={() => this.datePicker.focus()}>
               <ActionDateRange />
            </IconButton>

            <div style={{width:"0px", height:"0px", marginLeft:datePickerMargin}}>
               <DatePicker
                  id="dataPicker"
                  floatingLabelText={''}
                  value={this.state.selectedDate}
                  errorText={''}
                  disabled={false}
                  formatDate={date => { return format(date, 'MM/DD/YYYY') } }
                  autoOk
                  container="inline"
                  fullWidth
                  onChange={this.handleChangeDatePicker}
                  ref={c => {
                     this.datePicker = c
                  }}
               />
            </div>
         </div>
      )
   }

}

from mui-x.

isaackearl avatar isaackearl commented on May 16, 2024

I agree it is also a bit difficult to pick year if you are using this for entering something like a date of birth. Being able to type in text would help this.

from mui-x.

mbrookes avatar mbrookes commented on May 16, 2024

@nathanmarks A user could created their own version of DatePicker without TextField using DatePickerDialog or Calendar.

from mui-x.

nathanmarks avatar nathanmarks commented on May 16, 2024

@mbrookes If that's the case, that sounds like the best option for specific use cases such as this.

from mui-x.

canastro avatar canastro commented on May 16, 2024

@nathanmarks is this a specific use case? Its a input field that the native implementation allows for keyboard input. IMHO, this is the normal use case for any input component on a browser.

from mui-x.

mbrookes avatar mbrookes commented on May 16, 2024

it is also a bit difficult to pick year if you are using this for entering something like a date of birth.

@isaackearl The UX is poor (per MD spec), but if you click on the year in the date-display you get a list of years to choose from.

from mui-x.

isaackearl avatar isaackearl commented on May 16, 2024

@mbrookes Ah yes I am aware you can do that, but I still think it is not intuitive or easy to select a date quickly. I am having user tests on an application I've created and every tester has noted that it is difficult to use the datepicker for selecting date of birth. It does look nice though.

from mui-x.

nathanmarks avatar nathanmarks commented on May 16, 2024

@isaackearl @mbrookes

The spec recommends using segmented dropdowns when the UX is more appropriate

from mui-x.

mbrookes avatar mbrookes commented on May 16, 2024

@nathanmarks Yes - I said the same thing to @tintin1343 on the inline timepicker PR, as I think the current datepicker inline is not good UX, and we should deprecate it in favour of something built with dropdowns/

from mui-x.

ddeath avatar ddeath commented on May 16, 2024

So your suggestion is to be able choose which type of input method user prefer? Basically add one property which would indicate it?
If so I would like to create PR for TimePicker....

from mui-x.

rigelk avatar rigelk commented on May 16, 2024

@ddeath have you made any progress towards this?

from mui-x.

ddeath avatar ddeath commented on May 16, 2024

@rigelk Hi not at all, I am waiting for response from team member, I do not want to do something what will be thrown away... so I switched to select boxes as probably everybody here.

from mui-x.

canastro avatar canastro commented on May 16, 2024

@rigelk in the meanwhile I created a component in my project with something like this:

            <div className="acc-datepicker" style={styles.root}>
                <TextField
                    id={`${id}textfield`}
                    type="date"
                    ref="datepickerInternalInput"
                    value={this.state.dateStr}
                    errorText={this._buildErrors(id, allErrors)}
                    floatingLabelFixed={true}
                    floatingLabelText={floatingLabelText}
                    onBlur={this.isValid}
                    onChange={this._handleChange} />

                <ActionDateRange
                    onClick={this._onActionClicked}
                    color={cyan500}
                    style={styles.icon} />

                <DatePicker
                    ref="accDatePicker"
                    value={this.state.date}
                    onChange={this._handleChange} />
            </div>

The user can either write in the TextField of type date or click in the icon ActionDateRange to open the material-ui datepicker.

from mui-x.

rigelk avatar rigelk commented on May 16, 2024

@canastro I had not really thought about using a type="date" input. However it is not supported (shown) by Firefox…

from mui-x.

rigelk avatar rigelk commented on May 16, 2024

@markau could you point to that other implementation? Maybe we could adapt it and bring it here.

from mui-x.

markau avatar markau commented on May 16, 2024

@rigelk, it seems I was mistaken saying it is Material implementation, but it does use React.

https://grommet.github.io/docs/date-time

from mui-x.

aralot avatar aralot commented on May 16, 2024

here is my implementation: mui/material-ui#5677

I would appreciate any feedback

from mui-x.

caesay avatar caesay commented on May 16, 2024

Also mui/material-ui#5030 has been open with a working and production tested implementation since August

from mui-x.

dasloss avatar dasloss commented on May 16, 2024

Yeah, any expectation when mui/material-ui#5030 will be closed? We are eager to use it.

from mui-x.

reinartg avatar reinartg commented on May 16, 2024

@barbalex, I am trying to get your example to work in a react typescript project, but getting error on datePicker ( Property 'datePicker' does not exist on type 'MyDatePicker') used in StyledFontIcon's onClick method and DatePicker's ref method. Would you know how to solve this?

Could you also make a standalone react typescript component for your datepicker?

from mui-x.

barbalex avatar barbalex commented on May 16, 2024

@reinartg

Sorry: I use flow. I have never worked with typescript.

Property 'datePicker' is created by a ref (https://github.com/barbalex/apf2/blob/master/src/components/shared/DateFieldWithPicker.js#L168-L171).

You may have not created the ref properly.

I usually use functions for my components. In this case I use a class because ref's usually don't work in functions.

Maybe it does not work for you because you are not using a class?

from mui-x.

barbalex avatar barbalex commented on May 16, 2024

@reinartg it is kind of funny that flow also does not recognize the property created by a ref which is why I had to find a way to quiet it's complaints:

https://github.com/barbalex/apf2/blob/master/src/components/shared/DateFieldWithPicker.js#L169
https://github.com/barbalex/apf2/blob/master/.flowconfig#L9

from mui-x.

oliviertassinari avatar oliviertassinari commented on May 16, 2024

Closing for mui/material-ui#4787

from mui-x.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.