Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
What / Issue : while 'All day' is localized (can be translated by an admin on the site), the ( ) around it are hardcoded.
To get rid of them, I had to modify the file: date/date_all_day/date_all_day.module
function theme_date_all_day_label() {
return '' . t('All day', array(), array('context' => 'datetime')) . '';
The original was :
function theme_date_all_day_label() {
return '(' . t('All day', array(), array('context' => 'datetime')) . ')';
It is not a major issue, but ideally that should be easier to theme. In the previous version, I simply modified it using the locale.module in core Drupal. Alternatively, if they have a CSS style, they could be made invisible, but that would be a lot of bytes for only two ().
Comment | File | Size | Author |
---|---|---|---|
#5 | date-n2894759-5.patch | 494 bytes | DamienMcKenna |
| |||
#4 | all_day_the-2894759-4.patch | 500 bytes | Priyanka.addweb |
|
Comments
Comment #2
hiramanpatil CreditAttribution: hiramanpatil at Clarion Technologies commentedComment #3
hiramanpatil CreditAttribution: hiramanpatil at Clarion Technologies commentedI have created patch. Please check and review.
Comment #4
Priyanka.addweb CreditAttribution: Priyanka.addweb at AddWeb Solution Pvt. Ltd. commentedHere is the patch for the solution. The patch includes solution from the original comment.
Comment #5
DamienMcKennaWould it not be better to move the parentheses inside the string? Also the empty strings are no longer needed.
Comment #6
steinmb CreditAttribution: steinmb as a volunteer commentedComment #8
DamienMcKennaCommitted. Thanks!