Project

General

Profile

Actions
Copy link

Architecture #2141

closed

Better abstraction to factorize popup

Added by Arthur ANGLADE over 12 years ago. Updated about 12 years ago.

Status:
Released
Priority:
N/A
Assignee:
Arthur ANGLADE
Category:
Web - UI & UX
Target version:
2.4.0~alpha2
Pull Request:
Effort required:
Name check:
Fix check:
Regression:

Description

using a more generic function (by defining a default maxHeight and maxWidth in the createPopup function) , more Popups and Popup timeouts can be factorized

Actions
Copy link
 #1

Updated by Arthur ANGLADE over 12 years ago

  • Status changed from New to Pending technical review
  • % Done changed from 0 to 100
Actions
Copy link
 #2

Updated by Nicolas CHARLES about 12 years ago

  • Status changed from Pending technical review to 10

This seems valid !

Actions
Copy link
 #3

Updated by Jonathan CLARKE about 12 years ago

  • Status changed from 10 to Released

Closing these "Architecture" issues without a functional review - this step of the workflow has no sense here, there is no functional impact on code architecture changes (or there shouldn't be anyway!).

I think we should remove this step from the workflow for Architecture issues.

Actions
Copy link

Also available in: Atom PDF