Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Again a problem with spaces... #180

Closed
yourchoice opened this issue Feb 10, 2016 · 6 comments
Closed

Again a problem with spaces... #180

yourchoice opened this issue Feb 10, 2016 · 6 comments

Comments

@yourchoice
Copy link

I have this sequence
...
[{#variable#|mb_trim}]
...

Before last ] is added a space.

Please see and #177 . This issue is closed, but the problem still persists.

@yourchoice
Copy link
Author

Seems that after a smarty var is added a space. This is very ugly if we think to the forms... input int values validation.

Starting with version 3.1.28 seems to be a lot of bugs... Now i come back to the 3.1.27 version.

@uwetews
Copy link
Contributor

uwetews commented Feb 10, 2016

Can you give some full example where you still see the problem.
Please note there have been several commits for this problem, because I found myself cases where it did not work. 47237eb was the final commit for this problem

@yourchoice
Copy link
Author

Hi,

This is the example:

{strip}
{$text="Text"}
<span>#</span>{$text}<span>#</span>
{/strip}

The output with last dev. smarty version is : #Text #
After word "Text" there is a space

@yourchoice
Copy link
Author

Another situation that doesn't work is:

{strip}
<span>#</span> {'Text'}
{/strip}

The output is: #Text .

In this case, between # and Text , the space is removed.

PS: sorry, I started with issue #177, and here I forgot to add {strip} {/strip} to be more explicit (just Ii edited my posts).

@uwetews
Copy link
Contributor

uwetews commented Feb 10, 2016

{strip} is really tricky with all the different cases

Fix is in the master branch.

@uwetews uwetews closed this as completed Feb 10, 2016
@yourchoice
Copy link
Author

Again problem, I checked with dev /40 version

{strip}<i class="fa fa-map-marker"></i> <a href="#">Test</a>{/strip}

This not let the space between <i> and <a> tags.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants