Hi Hackers,
Here is an updated patch which solves the problem of more large query which
deals fine with zoom factor as well .
FYI: During testing it was found that the previous patch was only working
for mid sized query and not more large complex query due to zoom factor
issue.
Please review the new patch.
PFA.
On Tue, Apr 21, 2020 at 1:32 PM Akshay Joshi <akshay.jo...@enterprisedb.com>
wrote:
> Thanks, patch applied.
>
> On Mon, Apr 20, 2020 at 8:05 PM Yogesh Jain <yogesh.j...@enterprisedb.com>
> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch to fix the rightmost & bottom tooltip crop issues
>> in the explain query plan.
>>
>> FYI : While explaining a large query if we hover the rightmost or bottom
>> icon the tooltip is cropped & partially visible, which is fixed in this
>> patch.
>>
>> Please review.
>> PFA.
>>
>> --
>> Regards,
>> Yogesh Jain
>> 8982696654
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>
--
Regards,
Yogesh Jain
8982696654
diff --git a/web/pgadmin/misc/static/explain/js/explain.js b/web/pgadmin/misc/static/explain/js/explain.js
index 21167775d..935f2815e 100644
--- a/web/pgadmin/misc/static/explain/js/explain.js
+++ b/web/pgadmin/misc/static/explain/js/explain.js
@@ -994,8 +994,8 @@ define('pgadmin.misc.explain', [
toolTipX = toolTipX < 0 ? 0 : (toolTipX);
toolTipY = toolTipY < 0 ? 0 : (toolTipY);
- toolTipX = toolTipX > graphContainer.width()-toolTipContainer[0].clientWidth ? graphContainer.width()-(toolTipContainer[0].clientWidth*zoomFactor) : toolTipX;
- toolTipY = toolTipY > graphContainer.height()-toolTipContainer[0].clientHeight ? graphContainer.height()-(toolTipContainer[0].clientWidth*zoomFactor) : toolTipY;
+ toolTipX = toolTipX > graphContainer.width() - toolTipContainer[0].clientWidth ? toolTipX - (toolTipContainer[0].clientWidth+(pWIDTH* zoomFactor)) : toolTipX;
+ toolTipY = toolTipY > graphContainer.height() - toolTipContainer[0].clientHeight ? graphContainer.height() - toolTipContainer[0].clientHeight : toolTipY;
// Show toolTip at respective x,y coordinates
toolTipContainer.css({
