Closed (fixed)
Project:
Commerce Point of Sale (POS)
Version:
8.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
13 Nov 2017 at 20:59 UTC
Updated:
3 Jan 2018 at 23:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shabana.navas commentedComment #3
shabana.navas commentedThere's no shame in working on low-hanging fruit.
Comment #4
Aanal.addweb commented@shabana.navas, Thanks for Providing a fruitful patch, It works well for me & after applying the patch it shows added text with the link and after adding the register, it shows me the register dropdown with a Registered name.
Comment #5
alexpottHow about using
drupal_set_message(MESSAGE_TEXT, 'error');I think this is an error and using drupaL_set_message() will appropriately style the message.
Also this is not the best way to put a link into a message because the context and link text can't be translated. Something like this would work better:
This way the translator can translate everything in context and it is easier for right-to-left languages too.
Comment #6
shabana.navas commentedComment #7
sorabh.v6Working on it.
Comment #8
sorabh.v6Patchfile and interdiff posted with the change suggested in #5. Please review.
Thanks
Comment #9
smccabe commentedThanks all, committed