Skip to content

Fix: #12029 Long CRS name overflows in dropdown selector in CRS Selector#12030

Merged
tdipisa merged 3 commits intogeosolutions-it:masterfrom
anup39:fix_lng_crs_name_12029
Mar 3, 2026
Merged

Fix: #12029 Long CRS name overflows in dropdown selector in CRS Selector#12030
tdipisa merged 3 commits intogeosolutions-it:masterfrom
anup39:fix_lng_crs_name_12029

Conversation

@anup39
Copy link
Copy Markdown
Member

@anup39 anup39 commented Mar 2, 2026

Description

This PR fix long CRS name overflows in the CRS Selector.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

Issue

What is the current behavior?

#12029

What is the new behavior?

CRS name should be ellipsis.

Screenshot 2026-03-02 at 12 38 12 Screenshot 2026-03-02 at 12 38 34

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

Other useful information

@anup39 anup39 added this to the 2026.01.00 milestone Mar 2, 2026
@cla-bot cla-bot Bot added the CLA Ready label Mar 2, 2026
@anup39 anup39 linked an issue Mar 2, 2026 that may be closed by this pull request
1 task
@tdipisa tdipisa requested review from allyoucanmap and subashtiwari1010 and removed request for subashtiwari1010 March 2, 2026 11:55
button.dropdown-toggle {
padding: 0;
text-align: left;
max-width: 4.5rem;
Copy link
Copy Markdown
Contributor

@allyoucanmap allyoucanmap Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we try with

width: fit-content;
min-width: 4rem;

and remove max-width?

Copy link
Copy Markdown
Member Author

@anup39 anup39 Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@allyoucanmap I tried with the above style , also here is the short recording on how it looks now after the change.

crs_long_name.mp4

@anup39 anup39 requested a review from allyoucanmap March 3, 2026 14:24
@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Mar 3, 2026

@ElenaGallo please test in DEV and let us know if we can backport.

@tdipisa tdipisa merged commit 8d1bcf4 into geosolutions-it:master Mar 3, 2026
12 checks passed
@offtherailz
Copy link
Copy Markdown
Member

tdipisa pushed a commit that referenced this pull request Mar 11, 2026
…tor (#12030) (#12049)

* Fix: #12029 Long CRS name overflows in dropdown selector in CRS selector plugin

* fix maxwidth to allow display of long CRS name

* Added width to fit the content for crs name with min width of 4 rem

(cherry picked from commit 8d1bcf4)

Co-authored-by: Anup Dahal <adahal39@gmail.com>
@ElenaGallo
Copy link
Copy Markdown
Contributor

@anup39 When I select the customized CRS, the gray error page opens

crs.mp4

Use this context: https://dev-mapstore.geosolutionsgroup.com/mapstore/#/context/bvnb

@anup39
Copy link
Copy Markdown
Member Author

anup39 commented Mar 13, 2026

@ElenaGallo @tdipisa It seems like, I donot have permission to access the context you have shared. Can you provide me access to look into. Thanks
Screenshot 2026-03-13 at 12 35 11

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Mar 13, 2026

@ElenaGallo @tdipisa It seems like, I donot have permission to access the context you have shared. Can you provide me access to look into. Thanks

@anup39 I have granted permissions to the geosolutions group. Please try again.

@anup39
Copy link
Copy Markdown
Member Author

anup39 commented Mar 16, 2026

@anup39 When I select the customized CRS, the gray error page opens

crs.mp4
Use this context: https://dev-mapstore.geosolutionsgroup.com/mapstore/#/context/bvnb

@ElenaGallo @tdipisa After checking ,the error is due to the fact LongProjectionName:8950010 is not defined in projectionDefs in localConfig

https://github.com/geosolutions-it/mapstore-datadir/blob/DEV/configs/localConfig.json.patch

I think this is a different issue to be opened, and we need to check projectionDefs available before switching the projection.

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Mar 16, 2026

@anup39 you can raise a PR to the related repo for both DEV and QA branches:
https://github.com/geosolutions-it/mapstore-datadir

@anup39
Copy link
Copy Markdown
Member Author

anup39 commented Mar 17, 2026

@ElenaGallo @tdipisa After updating the localconfig https://github.com/geosolutions-it/mapstore-datadir/issues/54 , i tested in DEV and now it should work as expected .
image

@ElenaGallo
Copy link
Copy Markdown
Contributor

@anup39 opening this context (https://dev-mapstore.geosolutionsgroup.com/mapstore/#/context/bvnb
) in DEV and selecting the long CRS, the issue is still present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Long CRS name overflows dropdown selector.

5 participants