While I agree with @retornam that the suffix is probably unnecessary, I would prefer consistency. Currently just about all of our page objects in every project, including this one, use the suffix to identify variables that correspond to locators. I'm not sure that it's a good idea to just stop doing it now, and I think it's an even worse idea to go back and remove them. I know that nobody is suggesting doing the latter, but if we keep all the old code, and now just stop adding the suffix to new code, we end up with an inconsistent code base. I don't really see a lot of harm in keeping with the standard, so my vote would be to stick with it and add the suffix to whichever variables need it.