The only thing I can think of is maybe checking to see if the UI has reached the intended destination position before moving the UI. That way when the UI has reached the destination you won't need to keep calling clamp, which could impact performance.
Other than that, I can't think of anything else to add or suggest. I understood what the code snippet does and didn't find it unusual or weird, so I'd say the code looks fine.
