Related to himanshu-soni/ChatMessageView#21 (The default is right and…

[lolobosse]

… was not behaving as it should)

I guess that there was a bug because what I did was just to take to opposite of the left and it was working. Else right and long text wasn't working as mentioned in #21

[lolobosse]

@DaphnevanderVeeke If you look at the code completely you will see that the switch uses default for the "arrow right" case.

Then you can also see that the original "arrow right" case is very different from the "arrow left" case.

I mean it DIFFERENT (not the opposite like LEFT -> RIGHT).

What I did was to create the opposite of the left behaviour for the default of the switch.

Left was:

arrowRotation = 180;
arrowParams.addRule(ALIGN_PARENT_LEFT, TRUE);
arrowParams.setMargins(0, (int) arrowMargin, 0, (int) arrowMargin);
containerParams.addRule(RIGHT_OF, arrowImage.getId());

Whereas right was:

arrowRotation = 0;
containerParams.addRule(ALIGN_PARENT_LEFT, TRUE);
arrowParams.setMargins(0, (int) arrowMargin, 0, (int) arrowMargin);
arrowParams.addRule(RIGHT_OF, containerLayout.getId());

Which I replaced by the opposite of the original and working arrow-left case:

// Opposite of 180
arrowRotation = 0;
// Opposite of arrowParams.addRule(ALIGN_PARENT_LEFT, TRUE);
arrowParams.addRule(ALIGN_PARENT_RIGHT, TRUE);
// Margin should stay same
arrowParams.setMargins(0, (int) arrowMargin, 0, (int) arrowMargin);
// Opposite of arrowParams.addRule(RIGHT_OF, containerLayout.getId());
containerParams.addRule(LEFT_OF, arrowImage.getId());

@himanshu-soni Any plan to merge it soon?