Remove tracking of required fields

 






 

 


  Review Request #2352
  —
  Created March 17, 2023 and submitted  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  QuLogic
 









 
 

  pidgin/pidgin
 









 
 

  default
 









 
 

  
 









 
 

  2349, 2351
 






 




 

  

   Reviewers

  





 
 

  pidgin
 









 
 

  
 






 








 





  


 
 

  
The field required&filled status is now encapsulated in the valid property due to /r/2351.

This means that UIs no longer need to check that all required fields are filled.

So the page no longer needs to track required fields either.

 








  


 
 

  
Compiled and ran ninja test. Also opened Request Fields from Demo protocol, removed the required string or entered an invalid email, and confirmed the OK button disabled correctly.

 







 









 





  








 




 







   

    
    

    

    


    

      
Commits

      

        
      

    


    


      
Files