Automatic field detection is fine and good UX for password managers. What is bad is auto-fill without user action.